-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Pref > Library: add link to manual > settings files info #4367
Conversation
Nice. However this works, and this is a good start IMHO. Thank you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. What do the others think?
Yeah, discoverability can be improved. |
Yes, that will work hopefully better. I had a look to other applications:
So we are in good companion whatever we decide. |
so for now we can merge this? it's just adding the link, the rest is already there. |
I think the Help menu would be a better place. |
To clarify: Moving the entire section to another place will not happen in this PR. Merge? |
Eh, I'm not sure we should add even more text here; it's already quite wordy. I think it would be better to remove this whole section from the preferences and add a link to the Help menu. |
Those are currently three lines of text. Adding a link adds value now. Moving the entire block to the Help menu and formatting the dialog is out of scope here and will happen in another PR. |
Three lines of text is probably more than most users read already. |
Please just merge this. The fact that we're already 10 messages deep into a discussion on a PR this trivial makes no sense. This is bikeshedding at its best. The work has been done and we only need to press a single button to merge this. If instead we start arguing about marginally better approaches, we'll get nowhere, this won't get merged, contributors will stop contributing. |
I will not merge myself if there are objections, no matter if I consider them valid or not, no matter how ridiculous small the PR. |
I'm not sure that adding more text to a wall of text is a great idea but 🤷 |
Now, the directory and related info can be accessed from one place, and most support responses regarding log files, configs, mappings, database file etc. can be boiled down to
"Open the preferences, go to the Library page and scroll to the bottom"