webannoyances icon indicating copy to clipboard operation
webannoyances copied to clipboard

Google Image Search

Open RedDragonWebDesign opened this issue 3 years ago • 16 comments

I'm getting a floating header when I do Google Image Search.

https://www.google.com/search?q=table+of+c%2B%2B+and+java+collections&tbm=isch

2020-10-08_180015

I tried writing this filter, but it didn't like my combination of :has() and :style()

google.com##header:has([aria-label="Search by image"]):style(position: static !important;)

This less specific one works, but there's a second problem of a large whitespace above it.

google.com##header:style(position: static !important;)

High traffic website, so I figured I'd ask for help on this one.

RedDragonWebDesign avatar Oct 09 '20 01:10 RedDragonWebDesign

@RedDragonWebDesign Thanks for reporting! Are you able to test the filters below? if you watch closely on scrolldown/scrollup, there are lots of small changes the search bar makes which is why so many filters are necessary to keep the sizing consistent when going between scrolled down page and initial page.

I'm most concerned with how this could have unintended impact on the main google search function more-so than the google images since this is on the www.google.com domain and not limited to just image search.

www.google.com##header.Lj9fsd:style(position: absolute !important; top: 10px !important; box-shadow: none !important;)
www.google.com##.Lj9fsd.DU1Mzb .Si7Nvc:style(top: 0 !important;)
www.google.com##.Lj9fsd.DU1Mzb .F1hUFe:style(padding: 4px 28px 0 30px !important;)
www.google.com##.Lj9fsd.DU1Mzb .wQnou:style(height: 46px !important;)
www.google.com##.Lj9fsd.DU1Mzb .F7UV7d:style(height: 24px !important; width: 24px !important; background-size: 24px !important;)
www.google.com##.Lj9fsd.DU1Mzb .AWbbyf, .Lj9fsd.DU1Mzb .Aj0WOd, .Lj9fsd.DU1Mzb .lhkctf, .Lj9fsd.DU1Mzb .CzSCNb, .Lj9fsd.DU1Mzb .OWOjU, .Lj9fsd.DU1Mzb .AQxKeb:style(height: 24px !important; width: 24px !important;)
www.google.com##.Lj9fsd.DU1Mzb .JSAgYe:style(line-height: 34px !important;)
www.google.com##.Lj9fsd.DU1Mzb .zthhpf:style(height: 44px !important; align-items: normal !important;)

yourduskquibbles avatar Oct 14 '20 14:10 yourduskquibbles

@RedDragonWebDesign Have you had a chance to test this at all? I've had it in my own personal filters for about a week now and seeing no issues, thinking I will deploy soon unless you have noticed some issue on google search results that I haven't seen.

yourduskquibbles avatar Oct 26 '20 14:10 yourduskquibbles

Sorry for the delayed response. I'll throw these in my custom filters right now. I'll let you know if I notice anything.

edit: I did a quick initial test. Looks good so far.

RedDragonWebDesign avatar Oct 26 '20 14:10 RedDragonWebDesign

After scrolling down, there's some whitespace in the top right that might be worth fixing. Haven't noticed anything else.

image

RedDragonWebDesign avatar Nov 04 '20 21:11 RedDragonWebDesign

@RedDragonWebDesign Awesome, thanks for feedback I will see if I can fix this and then push these filters out to Web Annoyances Ultralist.

yourduskquibbles avatar Nov 05 '20 13:11 yourduskquibbles

@RedDragonWebDesign can you confirm adding these two additional filters fixes the white space and makes the right pane full height of your browser?

www.google.com##.l39u4d[style*="width:"]:style(top: 0 !important; height: 100% !important;) www.google.com##.WaWKOe[style*="opacity:"]:style(height: 100% !important;)

Full list of all filters to test:

www.google.com##.Lj9fsd.DU1Mzb .AWbbyf, .Lj9fsd.DU1Mzb .Aj0WOd, .Lj9fsd.DU1Mzb .lhkctf, .Lj9fsd.DU1Mzb .CzSCNb, .Lj9fsd.DU1Mzb .OWOjU, .Lj9fsd.DU1Mzb .AQxKeb:style(height: 24px !important; width: 24px !important;)
www.google.com##.Lj9fsd.DU1Mzb .F1hUFe:style(padding: 4px 28px 0 30px !important;)
www.google.com##.Lj9fsd.DU1Mzb .F7UV7d:style(height: 24px !important; width: 24px !important; background-size: 24px !important;)
www.google.com##.Lj9fsd.DU1Mzb .JSAgYe:style(line-height: 34px !important;)
www.google.com##.Lj9fsd.DU1Mzb .Si7Nvc:style(top: 0 !important;)
www.google.com##.Lj9fsd.DU1Mzb .wQnou:style(height: 46px !important;)
www.google.com##.Lj9fsd.DU1Mzb .zthhpf:style(height: 44px !important; align-items: normal !important;)
www.google.com##.WaWKOe[style*="opacity:"]:style(height: 100% !important;)
www.google.com##.l39u4d[style*="width:"]:style(top: 0 !important; height: 100% !important;)
www.google.com##header.Lj9fsd:style(position: absolute !important; top: 10px !important; box-shadow: none !important;)

yourduskquibbles avatar Nov 05 '20 13:11 yourduskquibbles

I did a quick test. Looks good to me!

RedDragonWebDesign avatar Nov 09 '20 13:11 RedDragonWebDesign

https://github.com/AdguardTeam/AdguardFilters/issues/67336#issuecomment-730471505

I think I will need to exclude these filters from android adguard mobile in the exclusion list but I don't have an Android device to test with, @RedDragonWebDesign any chance you have one to test with?

yourduskquibbles avatar Nov 19 '20 16:11 yourduskquibbles

Tested in Firefox For Android, with forced update of Web Annoyances Ultralist. Looks good.

I do see a problem in Windows 7+Chrome+1080p. If you click on an image, the vertical scroll bar for the right bar doesn't work.

Having 2 scroll bars is correct. But the left one is disabled. It should be scrollable, since there are useful page elements farther down (under where it says "Toolbox")

image

RedDragonWebDesign avatar Nov 23 '20 12:11 RedDragonWebDesign

Tested in Firefox For Android, with forced update of Web Annoyances Ultralist. Looks good.

I do see a problem in Windows 7+Chrome+1080p. If you click on an image, the vertical scroll bar for the right bar doesn't work.

Having 2 scroll bars is correct. But the left one is disabled. It should be scrollable, since there are useful page elements farther down (under where it says "Toolbox")

image

Thanks looks like others have reported this issue too, I will try to investigate fix for this today. Thanks for testing!

https://github.com/AdguardTeam/AdguardFilters/issues/68070#issuecomment-732807692

yourduskquibbles avatar Nov 24 '20 13:11 yourduskquibbles

@RedDragonWebDesign I think this should be fixed now, but leaving open in case you have time to confirm it is looking OK now. :)

yourduskquibbles avatar Nov 24 '20 13:11 yourduskquibbles

Hi. Actually thought this[screenshot] was initially a bug due to the Google Search Fixer addon for FF android. After a bit of investigating, I found it to be caused by the Web Annoyances filter alone. It goes bonkers only on the image result page. Not faced an issue anywhere else.

98811240-f362b400-2439-11eb-9b96-10b240b8d420

axeltrinidade avatar Nov 24 '20 21:11 axeltrinidade

@yourduskquibbles Looks good. Thanks for your time on this.

Looks like @axeltrinidade posted something in this thread too. Although that doesn't happen on my Firefox for Android. Alex, any idea what the difference is? Does that only happen when you also have your Google Search Fixer plugin turned on? Are you able to use the logger to narrow it down to a specific filter?

2020-11-24 19 10 25

RedDragonWebDesign avatar Nov 25 '20 03:11 RedDragonWebDesign

Yes, it's a compatibility issue with Google Search Fixer. Not too much of a worry I think, because this annoyance has actually pushed me towards finally trying ddg!

axeltrinidade avatar Nov 25 '20 05:11 axeltrinidade

Yes, it's a compatibility issue with Google Search Fixer. Not too much of a worry I think, because this annoyance has actually pushed me towards finally trying ddg!

@axeltrinidade So you confirm that if Google Search Fixer is disabled and uBO with Web Annoyances Ultralist is only plugin enabled this does not happen? If no issue with only uBO I'm guessing there are some DOM element being removed or set to display: none by Google Search Fixer that Web Annoyances Ultralist is also setting cosmetic properties on. It's possible if I can figure out what element is being set to not display by Google Search Fixer I could add the display: block property to the filter in Web Annoyances Ultralist and then it would be a race condition to see which plugin gets to set the property first.

Hard for me to really investigate that without an Android device to test with though and go looking through the Web Inspector

yourduskquibbles avatar Nov 25 '20 14:11 yourduskquibbles

So you confirm that if Google Search Fixer is disabled and uBO with Web Annoyances Ultralist is only plugin enabled this does not happen?

Yes. Like I said, no worries or urgency. Maybe for the long run, would be better for the list to be more compatible.

axeltrinidade avatar Nov 25 '20 14:11 axeltrinidade