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

[fix] Add current location to location stack from follow link popup #3198

Merged
merged 1 commit into from
Sep 10, 2017

Conversation

Frenzie
Copy link
Member

@Frenzie Frenzie commented Sep 10, 2017

See #3190 (comment)

@robert00s Does this also solve the other issue you noticed?

@robert00s
Copy link
Contributor

I think you mean @poire-z :)

@poire-z
Copy link
Contributor

poire-z commented Sep 10, 2017

:)
Nope, it doesn't.
But the simple fix for this other issue is to replace (just before self.ui.link:onGotoLink()) your UIManager:close(self) with self:onClose() (which calls UIManager:close(self) + clear the highlight).

(Confirmed the first issue is solved.)

* Add current location to location stack from follow link popup
  See koreader#3190 (comment)
* Word highlight not removed
* "Follow Link" not translated
* Typo in docs
@Frenzie Frenzie merged commit 7310937 into koreader:master Sep 10, 2017
@Frenzie Frenzie deleted the fix_location_stack branch September 10, 2017 12:26
@poire-z
Copy link
Contributor

poire-z commented Sep 10, 2017

Oups, just noticed there's another one that need similar treatment:

UIManager:close(self.highlight_dialog)

to be replaced with: self:onClose()

@Frenzie
Copy link
Member Author

Frenzie commented Sep 10, 2017

Thanks! :-)

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.

None yet

3 participants