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.

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

View image button no longer visible on CHROME #254

Closed
BriggsIDP opened this issue Jun 9, 2022 · 21 comments
Closed

View image button no longer visible on CHROME #254

BriggsIDP opened this issue Jun 9, 2022 · 21 comments
Labels
bug Something isn't working

Comments

@BriggsIDP
Copy link

Describe the bug
Last week the extensions view image button vanished and I've not be able to restore it though uninstalling and reinstalling.

Screenshots
image

System Information

  • Brave - Version 1.39.120 Chromium: 102.0.5005.99 (Official Build) (64-bit)
  • ViewImage - v3.6.3 (win x86-64) - en_GB
@BriggsIDP BriggsIDP added the bug Something isn't working label Jun 9, 2022
@urfausto
Copy link

urfausto commented Jun 9, 2022

same in vivaldi (chromium).

@funblaster22
Copy link
Contributor

I believe this has been fixed in 3.6.4 (Issue #242) and the Chrome web store just hasn't updated

@tsuliwaensis
Copy link

The UserScript stopped working also

@TheCodemonger
Copy link

TheCodemonger commented Jun 10, 2022

I did some investigation/debugging and discovered the following error being thrown by the extension in Chrome:
image

I was able to trace the likely cause of the nullref back to a switch statement responsible for populating the value of 'viewImageButtonText' - which has no default case. This remains true in the current version of the repo's main branch.

// Set the view image button text
    var viewImageButtonText;
    switch (version) {
        case VERSIONS.FEB18:
            viewImageButtonText = viewImageButton.querySelector('.Tl8XHc');
            break;
        case VERSIONS.JUL19:
            viewImageButtonText = viewImageButton.querySelector('.irc_ho');
            break;
        case VERSIONS.OCT19:
            viewImageButtonText = viewImageButton.querySelector('.pM4Snf, .KSvtLc');
            break;
    }

'viewImageButtonText' is later passed to 'localiseObject()' as the 'obj' arg.
While probably not the root cause of the issue and work needed for this update, can we please follow good coding practices and include default cases in switch statements - or at the very least provide a default value when declaring members?

This is my first dive into the Google page code relevant to this plugin, so I don't know if the above needs updating, but here are two examples of containers using the 'tvh9oe' class. The first is the one we want to target and additionally uses 'BIB1wf', others seem to use 'LRNLRb'.

<div jsname="CGzTgf" class="tvh9oe BIB1wf" data-tbnid="z4_uU0QB2pe-SM"></div>
<div jsname="R5S48d" class="tvh9oe LRNLRb" style="display: none; transform: translate3d(445px, 0px, 0px);" aria-hidden="true" data-tbnid="-VCM1w56w6u5VM"></div>

Any further down this rabbit hole and I might as well write my own replacement, lol.

Thank you for continuing to maintain this extension, @bijij!
I'll be patiently awaiting your fix/update.

@ner00
Copy link
Contributor

ner00 commented Jun 10, 2022

The UserScript stopped working also

Which one?

@ner00
Copy link
Contributor

ner00 commented Jun 10, 2022

I did some investigation/debugging and discovered the following error being thrown by the extension in Chrome

Well done. Coding practices are a good rule of thumb, worthless in this case.

Any further down this rabbit hole and I might as well write my own replacement, lol.

That would be reinventing the wheel, unless you make it dynamic enough to parse the structure regardless of Google's changes - as was the case. Part of the problem here is that people haven't been able or willing to read the history of #242

Thank you for continuing to maintain this extension, @bijij! I'll be patiently awaiting your fix/update.

There is a fix, and there is an update.

If I were @bijij I would leave at least on Chromium-related issue open since closing them perpetuates this problem of users not noticing that the issues has been brought up and solved, yet not available because... <rant>Google isn't very good at processing things manually (like vetting Chrome Web Store updates to extensions, although ironically letting the Play store become a cesspool of malware and spyware for mobile devices </rant>.

Back to the matter at hand, as mentioned by @funblaster22 this is indeed a duplicate of #242 and every other issue in between, as is this one. In less than 10 days, Google made 2 separate class name changes to their Google Images styling. The dev eventually fixed them but warned Chromium-based users (Chrome, etc) that the vetting process on the side of Google could take as long as 2 weeks, while Firefox would take a couple hours. Firefox users have had their situations solved for a good week now. Chromium-based users are depending on Google for approval or they would need to go through hoops to install an unsigned extension.

Now, while you guys are still having issues, and until the official extension gets vetted by Google, you can use the workaround I provided which builds on top of @bijij own UserScript which I updated and fixed when these issues started. If you want to use the workaround while the extension's update isn't officially available, read here: #242 (comment)

@tsuliwaensis
Copy link

The UserScript stopped working also

Which one?

I was referring to the one on this project, but I found the updated UserScript you posted on #242 that one works.

@trlkly
Copy link

trlkly commented Jun 14, 2022

@bijij It would be really nice if @ner00's script were linked on the front page (readme.md)--or better yet, uploaded to the update URL. It seems that the current link goes to version 3.5.0.

@bijij
Copy link
Owner

bijij commented Jun 14, 2022

Hi Folks, I'd been a bit under the weather since last week and hadn't taken a moment to look at GitHub, my apologies.

Firstly I'd like to thank @ner00 for taking it upon themselves to help me out there it's really been a great help.

So a few things:
1 - I definitely should have updated the user script, fixing the extension in the first place was my priority at the time, I was in a rush and forgot to, so apologies to everyone for that, I have now done this
2 - I have resubmitted the extension to google for review I am hoping this will be done before the end of the week, I am sorry that google's review process seems exponentially slower than Mozillas, (I see there is a PR to move to manifest v3, and I have a feeling doing so will speed up reviews, so fingers crossed)

I'm incredibly sorry for any inconvenience caused over the past couple weeks from this.

@bijij bijij changed the title View image button no longer visible View image button no longer visible on CHROME Jun 14, 2022
@bijij
Copy link
Owner

bijij commented Jun 14, 2022

I'll leave this issue open until google approves v3.6.4.

@firew4lker
Copy link

I have noticed that version 3.6.4 doesn;t work in firefox.

@ner00
Copy link
Contributor

ner00 commented Jun 14, 2022

It's working fine for me, but a few things to consider:

  1. If you previously used the UserScript with Tampermonkey, make sure to disable it or uninstall it since if both are used concurrently they will interfere with each other;
  2. Make sure to close the browser and return to the list of installed extensions, one time mine disappeared mysteriously after re-installation;
  3. Make sure you're not on a private tab (would need specific permissions for those);
  4. When in Google Images, press CTRL+F5 - or SHIFT+F5 in macOS - to reload the page/cache.

@firew4lker
Copy link

After several uninstallations and installations it seems to be ok.

@ner00
Copy link
Contributor

ner00 commented Jun 15, 2022

@bijij has gotten Google to approve v3.6.5 update, it's working perfectly afaik, I think this can now be closed.

@TeamRainless
Copy link

@bijij has gotten Google to approve v3.6.5 update, it's working perfectly afaik, I think this can now be closed.

Umm... NO.

March 2023. Not working again. Looks exactly like the photo in the OP.

@ner00
Copy link
Contributor

ner00 commented Mar 3, 2023

March 2023. Not working again. Looks exactly like the photo in the OP.

Umm... WORKS FINE.

Has been, for months, clearly.

chrome_eHOk7oYFUg

@TeamRainless
Copy link

TeamRainless commented Mar 4, 2023 via email

@bijij
Copy link
Owner

bijij commented Mar 6, 2023

Likely this is another A:B test from google.
Unfortunately the extension is still working for me, so I can't fix it.

PR's welcome

@vladimirgluten
Copy link

Likely this is another A:B test from google.

This is extremely weird because I just tried it on my 2nd google account and I do have the view image button, but if I use my main account it's not there. Now I want to know what the difference between the 2 accounts is!

@vladimirgluten
Copy link

And now even weirder, it just started working again using my main google account. I think I just needed to complain hard enough!

@TeamRainless
Copy link

TeamRainless commented Mar 10, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests