muckSponge / MaterialFox

A Material Design-inspired userChrome.css theme for Firefox
MIT License
1.94k stars 149 forks source link

URL Address Text has incorrect vertical padding while focused after opening new tab. #324

Open HomemadeToast57 opened 2 years ago

HomemadeToast57 commented 2 years ago

Sanity checks (must complete)

Describe the bug The vertical padding (specifically top padding) of the URL address text is incorrect within its container when you open or switch to a new tab and have focus on the search bar.

To Reproduce Steps to reproduce the behavior:

  1. Install MaterialFox chrome folder and user.js to profile root directory
  2. Restart Firefox
  3. Open new tab or click on address bar -> switch tabs -> return back to original tab (make sure you are not in the focus mode where it shows you autocomplete suggestions list)

Screenshot

Screen Shot 2022-07-24 at 3 08 47 PM

System info

Additional context I have a fix. It's a one line change that I will PR momentarily.

HomemadeToast57 commented 2 years ago

Since the time I posted this issue, I have fixed the bug and submitted a pull request to fix this. The pull request can be found here: #325.

Screenshot AFTER fix Look at the URL text inside the address bar...

Screen Shot 2022-07-24 at 3 34 29 PM

If you want to manually fix this and do not want to wait until they merge the pull request, change line #208 in chrome/urlbar/urlbar.css as shown below:

from
    padding: 0 !important;
to
    padding: 0 auto !important;

All this does is separates the horizontal and vertical padding and gives the top and bottom of the urlbar even padding (left and right padding is unchanged).

Hope this helps! :)