Skip to content

Feature: Display banner when recent items are turned off from File Explorer #10777

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

Merged

Conversation

jiejasonliu
Copy link
Contributor

Resolved / Related Issues

Details of Changes

  • show warning banner when setting is disabled in windows
  • slight refactor to empty recent files text

Validation

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

Screenshots (optional)

Files-5882.mp4

QuaintMako
QuaintMako previously approved these changes Dec 20, 2022
Copy link
Contributor

@QuaintMako QuaintMako left a comment

Choose a reason for hiding this comment

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

LGTM.

@yaira2
Copy link
Member

yaira2 commented Dec 20, 2022

Should the other message be hidden in this scenario?

@yaira2 yaira2 changed the title feat(recent-items): show warning when disabled in windows Feature: Display banner when recent items are turned off from File Explorer Dec 20, 2022
@jiejasonliu
Copy link
Contributor Author

Should the other message be hidden in this scenario?

i initially had it that way, but wasn't sure. it's been changed now.

Files-5882-hide-empty

Co-authored-by: yaira2 <39923744+yaira2@users.noreply.github.com>
@yaira2
Copy link
Member

yaira2 commented Dec 22, 2022

The banner seems to always be open

@jiejasonliu
Copy link
Contributor Author

The banner seems to always be open

that was the intention (if you disabled the setting), but what should be the better behavior?
we could make the banner closable and restore the text, but that raises questions on if it should be displayed once only

@yaira2
Copy link
Member

yaira2 commented Dec 22, 2022

It's showing for me even when the setting is enabled

@jiejasonliu
Copy link
Contributor Author

It's showing for me even when the setting is enabled

one thing note is you'll have to manually refresh the widget (or navigate away) for the change to update
that said, I cannot seem to reproduce this, but it'll likely be related to the registry values

@QuaintMako if you have a chance, could you try reproducing this on your machine?
you can modify File Explorer's recent files settings by clicking the ellipsis menu > Options as seen here:
image

@QuaintMako
Copy link
Contributor

@QuaintMako if you have a chance, could you try reproducing this on your machine? you can modify File Explorer's recent files settings by clicking the ellipsis menu > Options as seen here: image

Sure thing, will try to do it soon. If I don't report in two or three days, ping me again please!

@jiejasonliu
Copy link
Contributor Author

@yaira2 i believe the issue is resolved; it was due to an incorrect default value (registry keys don't appear unless changed once)

@yaira2 yaira2 added the changes requested Changes are needed for this pull request label Dec 28, 2022
Copy link
Member

@yaira2 yaira2 left a comment

Choose a reason for hiding this comment

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

LGTM

@yaira2 yaira2 removed the changes requested Changes are needed for this pull request label Dec 29, 2022
@yaira2 yaira2 added the ready to merge Pull requests that are approved and ready to merge label Dec 29, 2022
@yaira2 yaira2 merged commit 1c6e41b into files-community:main Dec 29, 2022
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.

Recent items shouldn't register elements when you have it disabled
3 participants