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.

Sign up for GitHub

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

Jump to bottom

FR: Option to prevent opening the internal browser for selected entries #141

Closed
danowar2k opened this issue Sep 14, 2017 · 5 comments
Closed

FR: Option to prevent opening the internal browser for selected entries #141

danowar2k opened this issue Sep 14, 2017 · 5 comments
Assignees
@martinrotter
Labels
Component-Message-List Component-Web-Browser Status-Fixed Ticket is resolved. Type-Enhancement This is request for brand new feature.
Milestone
3.5.5

Comments

@danowar2k
Copy link

Brief description of the issue.

Just a feature request. At the moment, when you select a feed article, the internal browser is opened at once to display the article. Could you add an option so that you can disable this function? So that it only opens on a keypress or double-click or whatever.
@martinrotter martinrotter self-assigned this Sep 15, 2017
@martinrotter
Copy link
Owner

Why would you want this feature? I do not get it. Is displaying of messages slow? Why exactly would anyone want this?

@danowar2k
Copy link
Author

Hm, maybe I'm alone with this. The internal browser seems really fast. It's just that when I don't use a function (the internal browser) I'd like to disable it to prevent unnecessary data download or CPU load.

But I guess I could always minify the internal browser window so that I'll probably forget about that. shrugs

@martinrotter
Copy link
Owner

I guess I can add some action to "Messages" menu, something like "Disable internal message preview", which could be binded to keyboard shortct, so that you could easily turn this feature on/off. Once disabled, no internal web viewer would be shown when cycling amongst messages.

What you think?

@danowar2k
Copy link
Author

That would be an absolutely sufficient implementation for what I'm thinking about. thumbsUp

martinrotter pushed a commit that referenced this issue Oct 23, 2017
Do some preparations for #141.
1b7c4f6
@martinrotter martinrotter added Component-Message-List Component-Web-Browser Status-Accepted I will eventually implement this. Type-Enhancement This is request for brand new feature. labels Oct 23, 2017
@martinrotter martinrotter added this to the 3.5.5 milestone Oct 23, 2017
@martinrotter martinrotter added Status-Fixed Ticket is resolved. and removed Status-Accepted I will eventually implement this. labels Oct 23, 2017
@martinrotter
Copy link
Owner

Test this once commit f6ed84b or newer commit gets compiled and let me know.

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

@martinrotter martinrotter

Labels
Component-Message-List Component-Web-Browser Status-Fixed Ticket is resolved. Type-Enhancement This is request for brand new feature.
Projects
None yet
Milestone
3.5.5
Development

No branches or pull requests
2 participants
@martinrotter @danowar2k