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

Remove uppercase on segment autocomplete #19777

Closed
wants to merge 2 commits into from
Closed

Remove uppercase on segment autocomplete #19777

wants to merge 2 commits into from

Conversation

mindbreaker
Copy link
Contributor

When the value is typed in a segment, it appears in capital letters. As a value, however, it is then taken over in such a way that only the first letter is written in capital letters.

snake14 and others added 2 commits September 27, 2022 13:17
…19738)

* Adding the abililty to exclude specific sites from the Vue site selector component.

* Adding more time to make sure that the select is loaded before trying to search.

* Changed things to filter the site on the server side.

* Added a few new test cases.

* Increasing amount of time for test.

* Updated the omnifixture.

* Fixing logic error in view.

* Removed collation from omnifixture since it was causing issues.

* Switching omnifixture back to 4.x-dev version.

* Making just the bare essential changes to make local test fixtures build correctly.

* Updating screenshot of API list.

* revert OmniFixture-dump to 4.x-dev

revert OmniFixture-dump to 4.x-dev

Co-authored-by: Peter <peter@innocraft.com>
@bx80 bx80 added this to the 5.0.0 milestone Sep 28, 2022
@bx80 bx80 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 Sep 28, 2022
@peterhashair peterhashair changed the base branch from 4.x-dev to 5.x-dev September 30, 2022 00:07
@peterhashair
Copy link
Contributor

@mindbreaker sorry to bother you, I am trying to merge this, but seems like there are some conflicts, do you mind fork a new PR based on 5.x-dev and then adding CSS changes? Then I can merge it.

@github-actions
Copy link
Contributor

This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers

@github-actions github-actions bot added the Stale The label used by the Close Stale Issues action label Oct 10, 2022
Copy link
Contributor

@peterhashair peterhashair left a comment

Choose a reason for hiding this comment

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

Looks good to me, we can add UI updates and UI tests later on. Ping @bx80

@peterhashair peterhashair linked an issue Nov 7, 2022 that may be closed by this pull request
@sgiehl
Copy link
Member

sgiehl commented Nov 9, 2022

This one has been replaced with #19958
Thanks @mindbreaker for your contribution

@sgiehl sgiehl closed this Nov 9, 2022
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 Stale The label used by the Close Stale Issues action
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Segment auto suggest capitalization
5 participants