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

Feature: Disable show more options menu item until shell items are loaded #11243

Merged
merged 14 commits into from
Feb 13, 2023

Conversation

hecksmosis
Copy link
Contributor

Resolved / Related Issues
Items resolved / related issues by this PR.

Validation
How did you test these changes?

  • Built and ran the app
  • Tested the changes for accessibility

Screenshots (optional)
Add screenshots here.

@yaira2
Copy link
Member

yaira2 commented Feb 12, 2023

@hecksmosis can you fix the merge conflicts?

@hecksmosis
Copy link
Contributor Author

Done

@ferrariofilippo
Copy link
Contributor

Works totally fine for folders and files, but not for sidebar items. Also, you should implement this for widgets as well.

@hishitetsu
Copy link
Member

One suggestion. Instead of simply disabling the menu while loading, how about changing the text to, for example, "Loading more options"? Simply disabling the menu may lead people to think that no additional options exist.

@yaira2
Copy link
Member

yaira2 commented Feb 12, 2023

One suggestion. Instead of simply disabling the menu while loading, how about changing the text to, for example, "Loading more options"? Simply disabling the menu may lead people to think that no additional options exist.

I agree

@hecksmosis
Copy link
Contributor Author

Probably

@yaira2
Copy link
Member

yaira2 commented Feb 13, 2023

I added the loading label for BaseLayout and the sidebar but it still needs to be added for the widgets.

@yaira2
Copy link
Member

yaira2 commented Feb 13, 2023

@hishitetsu with the loading label, should we still disable the menu or is the loading label clear enough?

@hishitetsu
Copy link
Member

I think it is still better to disable the menu.

@yaira2
Copy link
Member

yaira2 commented Feb 13, 2023

Widgets are done 👍

@yaira2
Copy link
Member

yaira2 commented Feb 13, 2023

One thing I noticed is that right clicking on Recycle Bin (from the Quick Access widget) or any file in the Recent Files widget causes the app to hang.

These are issues on main as well so LGTM

@yaira2 yaira2 merged commit 215c79a into files-community:main Feb 13, 2023
@yaira2 yaira2 added ready to merge Pull requests that are approved and ready to merge and removed needs - code review labels Feb 13, 2023
@hecksmosis
Copy link
Contributor Author

Thanks for finishing it, had a pretty busy couple of days and couldn't work on this

@hishitetsu
Copy link
Member

One thing I noticed is that right clicking on Recycle Bin (from the Quick Access widget) or any file in the Recent Files widget causes the app to hang.

I will submit a PR to fix these issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Pull requests that are approved and ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature: Make context menu entry "Show more options" always visible
4 participants