Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

made some changes in UI and improved visibility of page #1401

Merged
merged 2 commits into from
Sep 6, 2023
Merged

made some changes in UI and improved visibility of page #1401

merged 2 commits into from
Sep 6, 2023

Conversation

officialarmannqureshi
Copy link
Contributor

@officialarmannqureshi officialarmannqureshi commented Sep 1, 2023 •
edited

Here are the changes I've made:

  1. I've replaced the anchor tag with a button tag to remove the text decoration - underline property, making it more interactive.
  2. I also improved the visibility of the search button.

-> Change-1:
image

->Change-2:
image

@DonnieBLT
Copy link
Collaborator

Thank you!

@DonnieBLT DonnieBLT merged commit 9fddd9c into OWASP-BLT:main Sep 6, 2023
3 checks passed
@officialarmannqureshi
Copy link
Contributor Author

Thank you, @DonnieBLT @AtmegaBuzz , for your assistance in helping me create my first pull request for this project, which has been successfully merged. I'm excited to be a part of it and look forward to making more contributions in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DonnieBLT DonnieBLT DonnieBLT approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@officialarmannqureshi @DonnieBLT