MaterialFox icon indicating copy to clipboard operation
MaterialFox copied to clipboard

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

Open HomemadeToast57 opened this issue 3 years ago • 1 comments

Sanity checks (must complete)

  • [Y] I have read and followed the installation instructions in the README
  • [Y] I have not modified the userChrome.css file
  • [Y] I have tested the latest release for my Firefox version, or commit on master/beta branch (beta is for Firefox Beta only)

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

  • OS: MacOS
  • Firefox version: 103.0b9

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

HomemadeToast57 avatar Jul 24 '22 20:07 HomemadeToast57

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! :)

HomemadeToast57 avatar Jul 24 '22 20:07 HomemadeToast57