Skip to content

Added "Other resolution" button#44

Open
AHOHNMYC wants to merge 12 commits into
devunt:masterfrom
AHOHNMYC:master
Open

Added "Other resolution" button#44
AHOHNMYC wants to merge 12 commits into
devunt:masterfrom
AHOHNMYC:master

Conversation

@AHOHNMYC
Copy link
Copy Markdown
Contributor

  • Localization mechanism rewritten to support not only one string.
  • Added global variable linkToCurrentImage to store obvious link to current image
  • addButton() renamed to processElement()
  • addButton() now really only creates a button
  • Add "Other resolution" button.
    It takes a time to load a page because of loading other page in background.
    Maybe some kind of splashscreen can help user recognise that script is working.

Tested on Tampermonkey+Chromium.

* ! Localisation mechanism rewritten to support not only one string.
* Added global variable `linkToCurrentImage` to store obvuous link to current image
* Add "Other resolution" button.
It takes a time to load a page because of loading other page in background.
Maybe some kind of splashscreen can help user recognise that script is working.

Tested on Tampermonkey+Chromium.
@AHOHNMYC AHOHNMYC changed the title Add "Other resolution" button Added "Other resolution" button Feb 20, 2018
@AHOHNMYC
Copy link
Copy Markdown
Contributor Author

Splashscreen with Google's spinner added.

Attempt to close devunt#40 and devunt#50
Rewritten large image searching
Fixed localization selection
Removed `removeSplashscreen()` as unnecessary. Any error anyway will be logged in console
@AHOHNMYC
Copy link
Copy Markdown
Contributor Author

Closed #62, closed #63, closed #64, closed #65, probably #56 (#56 (comment))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant