Comments (13)
Problem is, that settings infrastructure currently does not provide any way to tell whether currently selected options are saved and if there is some option changed. Thus, when user would hit "Apply" and then "OK", settings would be actually saved twice. So, basically, it would work, but internally it is not ideal.
from rssguard.
Would saving twice yield any manifest problem then? Otherwise I'd assume the advantage in practice outweighs the not so nice technical implementation.
from rssguard.
No, it would not produce any bugs. I will add "Apply" and tweak the underlying implementation logic in the future.
from rssguard.
Hi Martin,
Just to add an idea how this usually works.
An user changes some options:
A. If he/she presses Apply, the settings are saved, the OK then simply closes the dialog
B. If he/she presses OK first, a new dialog appears asking for saving or discarding the changes made (may have Apply/Discard buttons).
C. If he/she presses Cancel, nothing will change (as usual)
from rssguard.
@xmstspider Thanks for clarification.
from rssguard.
Just to add an idea how this usually works.
Hm, are you sure?
Personally, I'm only aware of behaviour like this if it comes to closing modified documents.
But I've never seen any such thing regarding settings dialogues. The only behaviour I'm aware of is "OK" → save current settings and close dialogue without further query, "Apply" → apply current settings but leave dialogue open and "Cancel" → close dialogue discarding any changes that were made before. And tbh. I think this makes more sense.
from rssguard.
Yes, at least some settings centers like KDE Settings or similar in IDEs. They ask whenever you try to close the window or change the tab. It is more dummy-proof as you won't lose the changes when accidentally clicks on different tab/item/whatever...
But that's just an idea, it's up to you how do you want it ;-)
from rssguard.
It's true that System Settings of KDE are behaving that way. But these are a different kind of beast.
Just had another look: Buttons "OK" in the settings dialogues of LibreOffice, QupZilla, Otter Browser, FatRat, Gimp and SMPlayer as well as in the account settings of Thunderbird all save the current settings and close the dialogue without any querying. On the other hand I couldn't find a single application where hitting "OK" triggers some query. (A lot of applications are implementing some apply-on-change policy which is pretty stupid, IMO, and irrelevant for what we're discussing here anyway.)
So while this is certainly a matter of taste I really don't think the proposed behaviour can be called usual as far as settings dialogues of applications are concerned.
from rssguard.
Yes, at least some settings centers like KDE Settings or similar in IDEs
Exactly, I remembered KDE when reading your lines. It works like you said.
But on Windows, on the other hand, almost no SW I saw behaves like
B. If he/she presses OK first, a new dialog appears asking for saving or discarding the changes made (may have Apply/Discard buttons).
I would go more standard way:
OK
-> settings are saved (if changed), dialog is closed.APPLY
-> settings are save (if changed).
from rssguard.
No problem, it was just an idea :-)
from rssguard.
Will keep this as main feature for 3.3.3, which will be just bug fix release otherwise.
from rssguard.
Works as expected, thanks for implementing.
Changing the language at Settings - Language doesn't get applied by hitting Apply
but restarting RSS Guard only.
This is absolutely normal behaviour. But would a little popup informing users possibly make sense?
from rssguard.
Changing the language at Settings - Language doesn't get applied by hitting Apply but restarting RSS Guard only.
Yes the "restart warning" should popup, but maybe I missed it when transitioning the code to new approach. Will fix.
from rssguard.
Related Issues (20)
- [BUG]: UI unresponsive when switching feeds while sync is in progress HOT 8
- [BUG]: 4.6.0 fails to build if media player backends are disabled HOT 3
- [BUG]: 4.6.0 fails to build with cryptic errors when both media player backends are selected HOT 1
- [FR]: Enable Regex Queries to also have a local scope of search
- [BUG]: Mediaplayer tab opens each time I choose Open in external browser HOT 8
- [BUG]: "Non-Lite" Linux build identified as "Lite" as of v4.6.0 HOT 1
- [BUG]: Crashes on Linux if `LC_NUMERIC` is set to anything other than `C` HOT 3
- [BUG]: The "Browse" button in Media Player Settings does nothing
- Queue for podcasts HOT 5
- [BUG]: Tiny images HOT 3
- Source distfile weighs 58MB for 4.6.2 vs 22MB for previous versions HOT 3
- [BUG]: Flatpak Version number is incorrect HOT 9
- [FR]: Option to retain/exclude individual feeds from "remove read articles on app exit" feature HOT 3
- [BUG]: "Unknown File Format" When trying to play a Podcast HOT 21
- [FR]: Support for RSS-HUB HOT 11
- [BUG]: YT RSS feeds are being received with wrong date HOT 4
- [BUG]: Can't sync with Nextcloud HOT 2
- [FR]: Sort YouTube feeds by "published" date instead of "updated" date HOT 3
- [BUG]: Expanded folder gets collapsed after fetching new articles
- [BUG]: RSS Guard slows to a crawl when deleting items from a folder HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from rssguard.