Update views.md #663
Update views.md #663
Conversation
Update text to reflect the existence of the "Other" dropdown
| The functionality in darktable is separated into six different views: | ||
| The functionality in darktable is delivered through two main views and four subsidiary views: | ||
|
|
||
| **Main Views** |
There was a problem hiding this comment.
I think this should use markdown "header" syntax (with the # symbol) rather than just bold text. Additionally dtdocs has headings in lower case to match the style of the application UI.
| The functionality in darktable is delivered through two main views and four subsidiary views: | ||
|
|
||
| **Main Views** | ||
| <br>Switch between main views by clicking the view name at the top right of the window: |
There was a problem hiding this comment.
Please avoid html (remove the <br>). If you must have a new paragraph you can use a line feed in markdown. Similarly below.
|
|
||
| The currently active view is highlighted. | ||
|
|
||
| **Other Views** |
There was a problem hiding this comment.
Let's keep the terminology consistent. Are they subsidiary views or "other" views? Given that the application uses the term "other" I'd go with that, but I agree it makes the text harder to write. On reflection I'm not sure it's worth the effort to call them "main" and "other/subsidiary". Perhaps it's better to just add a line before the map/tethering sections saying something like "the following views are available by clicking on the 'other' label"
There was a problem hiding this comment.
Agree with using "other" instead of subsidiary.
The main reason for splitting out the "Other" views is that the UI already casts them as different/lesser by relegating them to the dropdown. So the manual should emphasise the ease of switching between lightable and darkroom which is where users will spend 90+% of their time.
Your call.
There was a problem hiding this comment.
On a practical note. I can make the corrections to the text in my fork alixtimix/dtdocs/patch-4
Once I commit those changes, how do I send the updated material back to you? A new push request? or what?
There was a problem hiding this comment.
If you update your branch that automatically updates the PR
There was a problem hiding this comment.
also note it isn't always called "other" -- when you change to the map view, then it is called "map", etc etc. So what it is called depends on what view you're currently in.
Revised initial Pull Request in lite of reviewer comments
|
Please use markdown headers, not bolded text.
…On July 18, 2024 10:38:54 AM PDT, Aliks ***@***.***> wrote:
@alikstimix commented on this pull request.
>
[lighttable](../../lighttable/_index.md)
: Manage images and collections.
[darkroom](../../darkroom/_index.md)
: Develop a single image.
+The currently active view is highlighted.
+
+**Other Views**
OK all done
--
Reply to this email directly or view it on GitHub:
#663 (comment)
You are receiving this because you are subscribed to this thread.
Message ID: ***@***.***>
|
|
This pull request has not had any activity in the past 60 days and will be closed in 365 days if not updated. Please verify it has no conflicts with the master branch and rebase if needed. Please add a comment if you need help or give permission to other people to finish your work. |
3 participants
Update text to reflect the existence of the "Other" dropdown