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

Fixes some UI issues in anonymise data #18413

Merged
merged 3 commits into from Dec 2, 2021
Merged

Fixes some UI issues in anonymise data #18413

merged 3 commits into from Dec 2, 2021

Conversation

sgiehl
Copy link
Member

@sgiehl sgiehl commented Nov 30, 2021

Description:

Found a few UI issues while testing some other stuff.

  • The remove icon (appearing when fields are added) was not really clickable as the input field had an higher z-index
  • The select fields were slightly misplaced

Note: There are currently a lot UI tests failing due to the fact, that the new version was tagged on another branch, but the changes are not yet included here...

Review

@sgiehl sgiehl added c: Design / UI For issues that impact Matomo's user interface or the design overall. Needs Review PRs that need a code review labels Nov 30, 2021
@sgiehl sgiehl added this to the 4.7.0 milestone Nov 30, 2021
@bx80 bx80 merged commit 5034993 into 4.x-dev Dec 2, 2021
@bx80 bx80 deleted the uifix branch December 2, 2021 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: Design / UI For issues that impact Matomo's user interface or the design overall. Needs Review PRs that need a code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants