Giter Club home page Giter Club logo

Comments (13)

martinrotter avatar martinrotter commented on May 22, 2024

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.

pmattern avatar pmattern commented on May 22, 2024

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.

martinrotter avatar martinrotter commented on May 22, 2024

No, it would not produce any bugs. I will add "Apply" and tweak the underlying implementation logic in the future.

from rssguard.

xmstspider avatar xmstspider commented on May 22, 2024

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.

martinrotter avatar martinrotter commented on May 22, 2024

@xmstspider Thanks for clarification.

from rssguard.

pmattern avatar pmattern commented on May 22, 2024

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.

xmstspider avatar xmstspider commented on May 22, 2024

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.

pmattern avatar pmattern commented on May 22, 2024

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.

martinrotter avatar martinrotter commented on May 22, 2024

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.

xmstspider avatar xmstspider commented on May 22, 2024

No problem, it was just an idea :-)

from rssguard.

martinrotter avatar martinrotter commented on May 22, 2024

Will keep this as main feature for 3.3.3, which will be just bug fix release otherwise.

from rssguard.

pmattern avatar pmattern commented on May 22, 2024

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.

martinrotter avatar martinrotter commented on May 22, 2024

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)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo 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.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.