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: Removed accent color from non-selected items in the column layout #10818

Merged
merged 19 commits into from
Dec 28, 2022

Conversation

marcelwgn
Copy link
Contributor

@marcelwgn marcelwgn commented Dec 22, 2022

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)
GIF of clicking around in Files with Column layout

@yaira2 yaira2 changed the title Fix accent color being shown on ColumnLayout if the element is not selected Feature: Removed accent color from non-selected items in the column's layout Dec 22, 2022
@yaira2 yaira2 changed the title Feature: Removed accent color from non-selected items in the column's layout Feature: Removed accent color from non-selected items in the column layout Dec 22, 2022
@yaira2
Copy link
Member

yaira2 commented Dec 22, 2022

Is the width of the item supposed to change?

This reverts commit efe15be.
@marcelwgn
Copy link
Contributor Author

This was the only way I got this to work right now :(

@yaira2
Copy link
Member

yaira2 commented Dec 22, 2022

Is it possible to hide the selection indicator and build our own into the item template?

@marcelwgn
Copy link
Contributor Author

It would be possible but a lot of work I think

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.

Very small formatting issues, aside LGTM.

src/Files.App/Views/LayoutModes/ColumnViewBase.xaml.cs Outdated Show resolved Hide resolved
src/Files.App/Views/LayoutModes/ColumnViewBase.xaml.cs Outdated Show resolved Hide resolved
src/Files.App/Views/LayoutModes/ColumnViewBrowser.xaml.cs Outdated Show resolved Hide resolved
QuaintMako
QuaintMako previously approved these changes Dec 26, 2022
@yaira2
Copy link
Member

yaira2 commented Dec 26, 2022

@chingucoding is keyboard navigation still an issue?

@marcelwgn
Copy link
Contributor Author

@yaira2 It seems to work now, but I would prefer if you would double check.

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 requested a review from gave92 December 27, 2022 18:55
@yaira2 yaira2 added ready to merge Pull requests that are approved and ready to merge and removed needs - code review labels Dec 28, 2022
@yaira2 yaira2 merged commit 950579f into files-community:main Dec 28, 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.

Feature: Remove accent color from non-selected items in the column's layout
4 participants