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

Mobile: Describe the plugin settings page #10207 #10511

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

danimnunes
Copy link
Contributor

@danimnunes danimnunes commented May 28, 2024

We chose not to proceed with the functionality topic related to GitHub
mirror because during our analysis of the code in the mobile app, we
did not fully comprehend how integrating with GitHub mirror would be
beneficial for the project.
We tried to fully understand how GitHub
mirror works and how it could be integrated into our project, but we
concluded that the part of the code related to plugins did not seem
suitable for using this functionality.
As we were unable to clearly
discern the objective or the benefits that GitHub mirror would bring
in this specific context, we decided to direct our efforts to areas of
the project where we could have a more immediate and understandable impact.

We focused on the area of plugin categories. First and foremost, it's important to note that a single plugin can belong to more than one category. However, upon reviewing the plugins page, we found that the immediate information available (without clicking on each plugin's description) seemed insufficient in terms of providing comprehensive details about the categories each plugin falls into. Therefore, the first functionality we implemented was on the plugin search page, displaying one of the categories to which each plugin belonged. If a plugin belonged to more than one category, we showed the symbol "+" as there wasn't enough space to list them all.

Additionally, if a plugin didn't belong to any specific category, we added "Other" to indicate its classification.
We also added a plugin search filter by category, and on each specific plugin page, we included all of its categories.

Here are some screenshots demonstrating what we implemented:

image

Plugin box:

image

Filtered by... :

image

Filtered by "productivity" example:

image

We chose not to proceed with the functionality topic related to GitHub
 mirror because during our analysis of the code in the mobile app, we
did not fully comprehend how integrating with GitHub mirror would be
 beneficial for the project. We tried to fully understand how GitHub
mirror works and how it could be integrated into our project, but we
 concluded that the part of the code related to plugins did not seem
 suitable for using this functionality. As we were unable to clearly
discern the objective or the benefits that GitHub mirror would bring
 in this specific context, we decided to direct our efforts to areas of
the project where we could have a more immediate and
understandable impact.
We focused on the area of plugin categories. First and foremost, it's
important to note that a single plugin can belong to more than one
category. However, upon reviewing the plugins page, we found that
the immediate information available (without clicking on each
plugin's description) seemed insufficient in terms of providing
comprehensive details about the categories each plugin falls into.
Therefore, the first functionality we implemented was on the plugin
search page, displaying one of the categories to which each plugin
belonged. If a plugin belonged to more than one category, we showed
the symbol "+" as there wasn't enough space to list them all. Each
plugin box had to remain concise, and we didn't want to completely
change the developers' project implementation. Additionally, if a
plugin didn't belong to any specific category, we added "Other" to
indicate its classification.
We also added a plugin search filter by category, and on each
specific plugin page, we included all of its categories.

Co-authored-by: Rafael Ribeiro <[email protected]>
@danimnunes danimnunes changed the title Implement: Mobile: Describe the plugin settings page #10207 Mobile: Describe the plugin settings page #10207 May 28, 2024
@personalizedrefrigerator
Copy link
Collaborator

personalizedrefrigerator commented May 29, 2024

Thank you for working on this!

I'm adding a link to #10207 to make it easier to find this PR from the issue.

A few comments:

  • A redesign of this settings page is being implemented in Mobile: Implement plugin screen redesign #10465 — this may lead to a large number of merge conflicts in the near future.
  • Consider posting an explanation of the design (perhaps including the above screenshots) on the Joplin Forum to gather community feedback.
    • It's recommended to discuss large changes on the Joplin Forum, particularly for UI-related changes. This helps determine whether the change is 1) likely to be accepted in its current form and 2) allows community members and contributors to suggest changes and ideas to the design before the implementation is finished.
  • I like the "filter by category" section by the search bar. However, it might be somewhat difficult to determine that these are categories.
    • Maybe this could be moved to a search filters dialog? (For example, filters button opens a dialog).

@personalizedrefrigerator personalizedrefrigerator added mobile All mobile platforms plugins Anything related to Joplin's plugin system enhancement Feature requests and code enhancements labels May 29, 2024
@danimnunes
Copy link
Contributor Author

Thank you for your feedback @personalizedrefrigerator !

To avoid the merge conflicts you mentioned due to the ongoing redesign of the settings page in Mobile, we will shift our focus towards creating the new filter as proposed.

We appreciate the suggestion and will work on implementing the filter by category within a search filters dialog. This approach should help in making the categories more apparent. Thank you for the example provided.

We'll also post an explanation of the design and gather community feedback on the Forum to ensure the changes are well-aligned with user expectations.

Thank you for your guidance!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Feature requests and code enhancements mobile All mobile platforms plugins Anything related to Joplin's plugin system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants