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

Does not initially indicate if too much text selected on Linux(?) #240

Open
Gitoffthelawn opened this issue Jan 24, 2020 · 12 comments
Open
Labels
bug Something isn't working

Comments

@Gitoffthelawn
Copy link
Contributor

Bug description

When creating a QR code from too much selected text, does not generate an error.

Steps to reproduce

  1. Go to any page with plenty of text.
  2. Press Ctrl+A
  3. Generate a QR code

Actual behavior

No error until you try to modify the text.

Expected behavior

Error that string is too long.

System

Operating system: Windows
Browser and version: Firefox 72.0.2
Add-on version: 1.6

@Gitoffthelawn Gitoffthelawn added the bug Something isn't working label Jan 24, 2020
@rugk
Copy link
Owner

rugk commented Jan 24, 2020

Cannot reproduce. If I use https://de.wikipedia.org/wiki/Wikipedia:Hauptseite and use the context menu to create the QR code, it does show me the error instantly.
Could you maybe specify the exact site, you've used?

grafik

Edit: This is a Windows device.

@Gitoffthelawn
Copy link
Contributor Author

That's funny, I tested it on Wikipedia too! :)

Here's the page I used: https://en.wikipedia.org/wiki/QR_code

I only got the error after I tried to modify the text.

I don't recall if I had the toolbar button on the in the overflow menu or not during that test. I have to go right now, otherwise I would check it for you.

@rugk
Copy link
Owner

rugk commented Jan 26, 2020

Okay, now can reproduce the issue with both Wikipedia pages.
Strangely, I could not with the dev version before. That is strange…

Maybe a timing issue/race condition?

On Fedora 31, Firefox 72.

@Gitoffthelawn
Copy link
Contributor Author

Yes, it could be a race condition. It happens every time for me, which is interesting that it doesn't do the same for you. I'm glad you mentioned that it happens on both pages for you.

@rugk
Copy link
Owner

rugk commented Jan 27, 2020

Yeah repdocuible on the device I mentioned above, but not on another Windows device. 🤔

@Gitoffthelawn
Copy link
Contributor Author

Same version of Windows and browser on each?

@rugk
Copy link
Owner

rugk commented Jan 28, 2020

Actually no, only reproducible on Linux (Fedora), not on Windows here. Same Firefox though. 🤔

@muxator
Copy link

muxator commented Jun 24, 2020

Hi,

confirmed on Ubuntu 20.04, Firefox 77.0.1, Offline QR Code Generator 1.7 (checked via about:telemetry#environment-data-tab_addons as I do not know any other way to see the installed version).

@Gitoffthelawn
Copy link
Contributor Author

@muxator Thank you for the confirmation. Mozilla has made it a little less obvious where to quickly find the version numbers for installed extensions. The trick is to go to about:addons, and then hover the cursor over the name of the extension. The extension's version should pop up in a tooltip.

@rugk
Copy link
Owner

rugk commented Jun 29, 2020

…or open the extension page details and scroll down to the bottom. There, everything is.

@rugk rugk changed the title Does not initially indicate if too much text selected Does not initially indicate if too much text selected on Linux(?) Jun 29, 2020
@Gitoffthelawn
Copy link
Contributor Author

@rugk That too! :)

@muxator
Copy link

muxator commented Jun 30, 2020

I checked the extension version using the three methods (yay). It's 1.7, indeed.

Here's a screen capture when replicating the issue. The context menu got cut off. The command I'm invoking is "QR code from selection".

Peek 2020-06-30 23-27

  • kubuntu 20.04
  • firefox 77.0.1
  • offline qr code 1.7

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

3 participants