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

fix: various issues #124

Merged

raisedadead merged 4 commits into freeCodeCamp:main from huyenltnguyen:fix/various-stuff

Apr 19, 2024

Member

huyenltnguyen commented Apr 19, 2024

Checklist:

I have read freeCodeCamp's contribution guidelines.
My pull request has a descriptive title (not a vague title like Update index.md)

This PR fixes the following issues:

Search box having nested focus line on focus
Advanced search button not having correct icon color
Buttons in the editor having a border (pointed out by Ilenia)
Disclosure widget text being unreadable on hover (closes Disclosure widget text is not readable on hover #110)

Screenshots

Issue	Before	After
Focus outline
Advanced search button
Editor buttons
Disclosure widget

huyenltnguyen added 4 commits

April 19, 2024 22:34


          fix: focus outline

4ed1b5d


          fix: editor buttons

839aeef


          fix: disclosure widget

81fc4b9


          fix: advanced search button

63dbfbe

huyenltnguyen requested a review from a team as a code owner

April 19, 2024 16:33

raisedadead approved these changes

View reviewed changes

raisedadead merged commit 8e9fbc8 into freeCodeCamp:main

1 check passed

huyenltnguyen deleted the fix/various-stuff branch

April 22, 2024 04:23

huyenltnguyen mentioned this pull request

fix: pad the search box evenly #127

Merged

2 tasks

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

Labels

None yet