Skip to content

Fixes the bad alignment issue on the missing dataview text#147433

Merged
stratoula merged 1 commit intoelastic:mainfrom
stratoula:fix-no-dateview-alignment
Dec 13, 2022
Merged

Fixes the bad alignment issue on the missing dataview text#147433
stratoula merged 1 commit intoelastic:mainfrom
stratoula:fix-no-dateview-alignment

Conversation

@stratoula
Copy link
Contributor

Summary

Closes #147392

Fixes the link alignment issue in both Visualize and Lens.
Screenshot 2022-12-13 at 2 11 24 PM

This has been caused due to an update on the EUI and the fact that the <RedirectAppLinks/> component adds its own style (flex which makes the link to be aligned on the left), This component is used in many places so I decided to not make css changes there as it might affect other places.

@stratoula stratoula added Feature:Visualizations Generic visualization features (in case no more specific feature label is available) Feature:Lens v8.7.0 release_note:fix backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes and removed release_note:fix labels Dec 13, 2022
@kibana-ci
Copy link

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
lens 1.3MB 1.3MB +21.0B
visualizations 267.4KB 267.4KB +21.0B
total +42.0B
Unknown metric groups

ESLint disabled in files

id before after diff
osquery 1 2 +1

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
fleet 60 66 +6
osquery 109 115 +6
securitySolution 445 451 +6
total +20

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
fleet 69 75 +6
osquery 110 117 +7
securitySolution 521 527 +6
total +21

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@stratoula stratoula added the Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// label Dec 13, 2022
@stratoula stratoula marked this pull request as ready for review December 13, 2022 13:19
@stratoula stratoula requested a review from a team as a code owner December 13, 2022 13:19
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-visualizations @elastic/kibana-visualizations-external (Team:Visualizations)

Copy link
Contributor

@dej611 dej611 left a comment

Choose a reason for hiding this comment

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

Tested locally on Safari 👍

@stratoula stratoula merged commit be95686 into elastic:main Dec 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting Feature:Lens Feature:Visualizations Generic visualization features (in case no more specific feature label is available) release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v8.7.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Lens] bad alignment on missing data view message

6 participants