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

Use icons for file dialog favourites #1186

Merged
merged 4 commits into from Jul 20, 2020
Merged

Use icons for file dialog favourites #1186

merged 4 commits into from Jul 20, 2020

Conversation

Jacalz
Copy link
Member

@Jacalz Jacalz commented Jul 12, 2020

Description:

This PR adds icons to the file picker favourites buttons to make it look a bit better and to make it possible for the user to know what a favourite points to without reading the full name of it. To make that happen, I had to add icons for download, computer and document.

image

Checklist:

  • Tests included.
  • Lint and formatter run with no errors.
  • Tests all pass.

Copy link
Member

@andydotxyz andydotxyz left a comment

Choose a reason for hiding this comment

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

This looks great.
For completeness shouldn't there be a drive icon for the windows drive list as well?

@Jacalz
Copy link
Member Author

Jacalz commented Jul 19, 2020

I added the drive icon that you requested. Should be fine for a re-review now :)

@Jacalz Jacalz requested a review from andydotxyz July 19, 2020 16:19
Copy link
Member

@andydotxyz andydotxyz left a comment

Choose a reason for hiding this comment

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

Nice one thanks :)

@Jacalz Jacalz merged commit 172bae0 into fyne-io:develop Jul 20, 2020
@Jacalz Jacalz deleted the fancy-file-favourites branch July 20, 2020 12:19
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.

None yet

2 participants