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

Extend docs to include macOs options access #217

Merged
merged 4 commits into from
Jan 15, 2024

Conversation

sambhavnoobcoder
Copy link
Contributor

@sambhavnoobcoder sambhavnoobcoder commented Jan 11, 2024

Summary

This is a…

  • Correction
  • Addition
  • Restructuring
  • Minor / simple change (like a typo)
  • Other

Reason for the Change

I was trying to address was how to access the plugins for Picard , which was discussed in the thread here : https://community.metabrainz.org/t/how-to-run-plugins-on-picard-as-a-developer/674175 .I think that clearly mentioning the access to plugins or options menu to MacOs users would be more intuitive and useful .

Jira Ticket (optional) - PICARD-2818

Description of the Change

I essentially added the path of action which would lead to the user reaching the plugins selection via the options menu in a MacOs environment , extending the guide to the windows based systems . adjoining with the pr are screenshots demonstrating the working of the pathway updated in the pr .

Screenshot 2024-01-12 at 2 15 11 AM

img 1 . Python --> Preferences

Screenshot 2024-01-12 at 2 15 54 AM

img 2. plugins selection

Additional Action Required

No Additional Action Required .

@rdswift rdswift requested a review from phw January 11, 2024 21:40
Copy link
Member

@phw phw left a comment

Choose a reason for hiding this comment

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

Thanks. A few corrections.

config/configuration.rst Outdated Show resolved Hide resolved
config/configuration.rst Outdated Show resolved Hide resolved
@phw phw requested a review from rdswift January 11, 2024 22:47
a new window with the option groups listed in a tree format on the left hand side, and the individual
settings on the right hand side. This is where the majority of Picard's customization is performed.
The option settings are found under the :menuselection:`"Options --> Options..."` item on the menu bar. Whereas in the MacOs ,the option
settings are found under the :menuselection:`"Python --> Preferences ..."` item on the menu bar . This will open a new window with
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't think there should be a space between "Preferences" and the three periods that follow. At some point, I plan to replace all occurrences of three periods with a proper ellipsis, so you can do that here if you like (but not necessary).

config/configuration.rst Outdated Show resolved Hide resolved
@sambhavnoobcoder
Copy link
Contributor Author

okay , adding this as a note seperately does make more sense . I'll make that change and put up the pr .I'll also make changes to the macOs part as well in it along with the smaller space changes .

@sambhavnoobcoder
Copy link
Contributor Author

@phw , @rdswift I have put up a new commit addressing the issues in the review . The PR addresses the following issues :

  • updated spelling of macOs to be consistent exactly like this throughout .
  • punctuation is now bar. and not bar ..
  • added the menu access information in a note just below the paragraph , instead of inside the paragraph using the .. note:: block.
  • removed the space between the 'Preferences' and the three dots . (I've kept the three periods for now , but I will gladly help @rdswift whenever he plans to replace them with the proper ellipsis throughout the document).

kindly review the new commit as well .

config/configuration.rst Outdated Show resolved Hide resolved
@sambhavnoobcoder
Copy link
Contributor Author

@phw I have added the change to guide the user running the downloaded version of the application to the options menu . as for the "MusicBrainz Picard" v/s "Picard" issue , its kind both in Mac actually . when the application starts, as long as the application is in the phase where it displays the warning dialog box , it says Picard in the bar , however once the files / folder are uploaded the bar starts saying MusicBrainz Picard . I am attaching the images for both the cases below :

Screenshot 2024-01-12 at 10 10 01 PM

img 1. illustration of the bar saying "Picard" when the app is first opened with the dialog box

Screenshot 2024-01-12 at 10 14 28 PM

img 2 . illustration of the bar saying "MusicBrainz Picard" once files are uploaded .

My last commit adds the guide with reference to MusicBrainz , however should you want me to change that , I'll gladly comply .

@rdswift rdswift changed the title extended docs to include MacOs options access Extend docs to include macOs options access Jan 14, 2024
format on the left hand side, and the individual settings on the right hand side. This is where the majority of Picard's
customization is performed.

.. note::
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is a really minor item, but for consistency can you please remove one of the spaces between ".." and "note::"? Thanks,

Suggested change
.. note::
.. note::

I think the rest of the content is okay, but I've never used a mac in my life, so I can't speak to the way it handles the menu bar.

@sambhavnoobcoder, looking at your original images again it appears that the "Options" link appears to the right of "MusicBrainz Picard". What happens if you click on this item? Does it provide a sub-menu with an item to open the option settings window? If so, what is the difference between this and using "MusicBrainz Picard --> Preferences..."?

Copy link
Member

Choose a reason for hiding this comment

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

Does it provide a sub-menu with an item to open the option settings window?

No, on macOS that item is under the menu with the application name. Hence this clarification

@sambhavnoobcoder
Copy link
Contributor Author

@rdswift , the option link that you are seeing seems to be housing other features like move files , rename files , save tags , etc . I am attaching a screenshot of the option tab below :

Screenshot 2024-01-15 at 4 49 18 PM

img . options tab in macOS

I have also fixed the note whitespacing that you mentioned in the review above . kindly review that as well and merge if all is okay .

Copy link
Member

@phw phw left a comment

Choose a reason for hiding this comment

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

Looks good, thanks a lot for both noticing the issue and the update.

@phw
Copy link
Member

phw commented Jan 15, 2024

CI reports some Sphinx incompatibility, but seems unrelated to this PR and rather is likely some issue with updated Python packages. Seems to be because of this recent update of https://pypi.org/project/sphinxcontrib-applehelp/1.0.8/ :

The sphinxcontrib.applehelp extension used by this project needs at least Sphinx v5.0; it therefore cannot be built with this version.

We probably should follow and update to use Sphinx 5.0. Not sure what it involves.

This will not block us merging the PR, though. But I'll have @rdswift take a final look on the changes here.

@phw
Copy link
Member

phw commented Jan 15, 2024

as for the "MusicBrainz Picard" v/s "Picard" issue , its kind both in Mac actually . when the application starts, as long as the application is in the phase where it displays the warning dialog box , it says Picard in the bar , however once the files / folder are uploaded the bar starts saying MusicBrainz Picard .

@sambhavnoobcoder This is actually interesting. I'd count this as a bug in Picard, but unsure how and even if this can be resolved (Qt takes care of most of the handling for the global menu bar). But maybe application name is updated later on or something. It might also be because of the dialog itself. The global menu bar is a bit special, as it can change with dialogs.

@sambhavnoobcoder
Copy link
Contributor Author

Looks good, thanks a lot for both noticing the issue and the update.

your welcome . in that case Ig once @rdswift ,looks at it , if its all fine we can merge the pr .

We probably should follow and update to use Sphinx 5.0. Not sure what it involves.

I think I may try to update to use Sphinx 5.0. Not sure what it involves either , but I'll give it a try and if I am able to do it , I'll raise a pr for the same .

I'd count this as a bug in Picard, but unsure how and even if this can be resolved

I'll see what can be done for the menu bar as well , and if I manage to make a breakthrough , I'll raise a ticket or pr for that as well .

@rdswift rdswift merged commit 5f139e3 into metabrainz:master Jan 15, 2024
1 check failed
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