Skip to content

use flex layout to be able shorten text to two lines #21304

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
merged 1 commit into from
Sep 27, 2023

Conversation

sgiehl
Copy link
Member

@sgiehl sgiehl commented Sep 26, 2023

Description:

follow up to #21247 to improve layout further

Review

@sgiehl sgiehl marked this pull request as ready for review September 26, 2023 11:09
@sgiehl sgiehl added this to the 5.0.0 milestone Sep 26, 2023
@sgiehl sgiehl added not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. Needs Review PRs that need a code review labels Sep 26, 2023
@sgiehl sgiehl requested a review from a team September 26, 2023 11:10
Copy link
Contributor

@michalkleiner michalkleiner left a comment

Choose a reason for hiding this comment

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

When testing 21247 locally I already had this branch checked out and the 'two lines max' mechanism works as expected.

@sgiehl sgiehl merged commit 3c1a7c2 into nodatalayout Sep 27, 2023
@sgiehl sgiehl deleted the nodatalayoutfix branch September 27, 2023 06:46
sgiehl added a commit that referenced this pull request Sep 29, 2023
* Allow custom loading message for widget loader

* apply new layout to header and footer of no data page

* implement layout of tracking method view

* layout improvements

* implement recommended method

* add hidden share button for later

* Refactor rendering into vue component

* implement details view for tracking methods

* implement url handling for tab selection

* show notifications in details only if it was detectd

* improve events and move some stuff to tagmanager

* some layout tweaks

* Consolidate EmptySite UI test suites into one

* move test to tag manager

* code cleanup

* removes some unused translations

* further adjustments to various method details

* implement method to allow recommendation overwrite

* allow selecting specific details view in css

* implement global list styling

* move tagmanager specific styling to plugin

* remove GA3 and GA4 notifications from tracking code screen

* remove no longer needed tests and fixtures

* ensure url parameter uses same case as before

* layout and code fixes

* improve global list style

* fix sorting

* updates expected UI test files

* submodule update

* update outdated css selectors

* avoid unnecessary screenshots in updater tests

* fix parameter type hint

* Move check for tracked data to vue component

instead of reusing the live widget

* don't show GA Import on others screen when plugin is available

* improve layout so it works with two line names

* use flex layout to be able shorten text to two lines (#21304)

* remove indention of test feature result

* fix sizing of code block elements in numbered lists

* Improve handling of CTA buttons

* adjust css

* submodule update
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Review PRs that need a code review not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org.
Development

Successfully merging this pull request may close these issues.

2 participants