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

Reorganize preferences dialog #447

Merged
merged 1 commit into from
Sep 5, 2021
Merged

Reorganize preferences dialog #447

merged 1 commit into from
Sep 5, 2021

Conversation

xiota
Copy link
Contributor

@xiota xiota commented Sep 4, 2021

Proposed reorganization of preferences.

screenshot-09-preferences

@xiota xiota changed the title Reorganize options in preferences dialog Reorganize preferences dialog Sep 4, 2021
@tsujan
Copy link
Member

tsujan commented Sep 4, 2021

Please:

  1. Don't touch settings.cpp! Changing groups is backward incompatible and we avoid that in LXQt. Moreover, there's no need to a correspondence between Preferences dialog and config file. The former is for the user, while the latter is for the app.
  2. Don't drop "by default". It means "when the app starts". The changes made to the main window are temporary and that's a feature we don't want to remove.

@xiota
Copy link
Contributor Author

xiota commented Sep 4, 2021

I updated the screenshot in the main description.

I put settings.cpp back to how it was before except that I moved ShowExifData to the Windows section. It was in the Thumbnails section before, and the people this change affects should be limited since it was added only recently.

I put "by default" back. I had removed it because it seemed redundant. The preferences dialog should naturally affect the default. It also appears repeated in multiple options right next to each other.

Is it possible to make a labeled frame that says "Startup defaults"? Then some of the redundancy could be reduced. (I'm looking at the Qt widgets, and it looks like QFrame doesn't have a label.)

#449 and #450 are small changes that don't affect the preferences dialog, if you want something small to look at next.

@tsujan
Copy link
Member

tsujan commented Sep 4, 2021

The preferences dialog should naturally affect the default.

When an option says, for example, "Show main toolbar", its unchecking should remove the toolbar immediately or after closing the Preferences dialog (preferably, the first). But when it says, "Show main toolbar by default", the user knows it changes a default behavior.

@tsujan
Copy link
Member

tsujan commented Sep 4, 2021

I tested it. I'm afraid it seems to me like too many tabs for a few options.

We need a third opinion. @stefonarch, would you please test it and see if you like its Preferences dialog?

@xiota
Copy link
Contributor Author

xiota commented Sep 4, 2021

Maybe Thumbnails could be combined with Image. Image will become crowded later though.

Or maybe Thumbnails could go with General.

@tsujan
Copy link
Member

tsujan commented Sep 4, 2021

Let's wait for @stefonarch!

EDIT: I asked @stefonarch's opinion on 5 tabs; didn't notice that they're 4 now.

Copy link
Member

@tsujan tsujan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is much better. Thanks!

@tsujan tsujan merged commit 92354eb into lxqt:master Sep 5, 2021
@xiota
Copy link
Contributor Author

xiota commented Sep 5, 2021

Glad you like this better. Let me know when you feel like looking at any other PRs. Also, what's your opinion of using Exiv2 instead of libexif or as a compile-time selection? Reason I'm thinking about it is Exiv2 supports more file types.

@xiota xiota deleted the 5B-rearrange branch September 5, 2021 23:21
@stefonarch
Copy link
Member

Let's wait for @stefonarch!

EDIT: I asked @stefonarch's opinion on 5 tabs; didn't notice that they're 4 now.

Sorry if I missed something, but it isn't easy to track so many GH notifications. Preferences look good now.

@tsujan
Copy link
Member

tsujan commented Sep 6, 2021

but it isn't easy to track so many GH notifications.

I use Thunderbird and filter LXQt mails, so that they go to an LXQt folder.

@stefonarch
Copy link
Member

but it isn't easy to track so many GH notifications.

I use Thunderbird and filter LXQt mails, so that they go to an LXQt folder.

I do so too, but when there are >30 in the morning I prefer to overfly the conversation on GH and looks I missed something. I thought about an additional filter for @stefonarch ...

@tsujan
Copy link
Member

tsujan commented Sep 6, 2021

... but when there are >30 in the morning...

It happens here too. Most of them are Weblate translation notifications — "Translations update from Weblate" as a filter might make it better for you.

@stefonarch
Copy link
Member

stefonarch commented Sep 6, 2021

I've a filter for them too, they doesn't get mixed with the rest. But last days just lxqt was quite busy :)

@xiota
Copy link
Contributor Author

xiota commented Sep 6, 2021

@stefonarch

But last days just lxqt was quite busy :)

If that was from me, I will slow down soon.

@tsujan
Copy link
Member

tsujan commented Sep 6, 2021

If that was from me,

No one needs to worry about notifications. Of course, it's good practice not to write everything that comes to one's mind as a comment but your comments haven't been of that sort.

Concise comments are always preferable. Rarely, some people write short stories, instead of reports.

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.

3 participants