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

fix(annotations): displaying annotation tooltips correctly #1406

Merged

Conversation

mfranzke
Copy link
Contributor

@mfranzke mfranzke commented Jan 2, 2022

We're currently facing problems especially on the "standalone" view of patterns (e.g. pages), in which the general pattern-lab.css isn't referenced, that includes the /packages/uikit-workshop/src/sass/scss/04-components/_annotations.scss file.

Summary of changes:

  • Hiding elements (in this case the annotation tooltips) in a modern way by the hidden attribute instead of style property manipulations.
  • Extracted parts of the annotations CSS code so that it could even also get imported into pattern-scaffolding.css (which we needed to convert to a SCSS file and transform that one as well).

Preview URL with the fixed version

"standalone" view of patterns (e.g. pages)

mfranzke and others added 30 commits April 24, 2020 16:56
missed to replace mustache initially and incorrect handlebars two times (both in package name and URL)
This reverts commit 7812878, reversing
changes made to 26db979.
This reverts commit 26db979, reversing
changes made to fc90750.
@mfranzke mfranzke marked this pull request as ready for review January 23, 2022 12:34
@JosefBredereck JosefBredereck merged commit 3f33ce5 into pattern-lab:dev Jan 29, 2022
@mfranzke mfranzke deleted the fix/annotations-tooltip-rendering branch January 29, 2022 19:38
@JosefBredereck
Copy link
Contributor

PR was released with v5.16.0

antonia-rose pushed a commit to quelltexterin/nemo-uikit-workshop that referenced this pull request Apr 12, 2023
…ab#1406)

* chore(docs): some typos

missed to replace mustache initially and incorrect handlebars two times (both in package name and URL)

* Revert "Merge pull request pattern-lab#2 from pattern-lab/dev"

This reverts commit 7812878, reversing
changes made to 26db979.

* Revert "Merge pull request pattern-lab#1 from pattern-lab/dev"

This reverts commit 26db979, reversing
changes made to fc90750.

* Revert "chore(docs): some typos"

This reverts commit 271dc8d.

* chore: some reformatting

* chore: formatting

* chore: formatting

This reverts commit 271dc8d.

* chore: formatting

* refactor: let's hide elements the modern way with the hidden attribute

* refactor: extracting annotations inside modal styles

* refactor: we won't need this as a generated CSS file

* Revert "Merge branch 'dev' into fix/annotations-tooltip-rendering"

This reverts commit 2c715ae, reversing
changes made to 5effb4f.

* refactor: adding some sample content for the annotations

and corrected the current approach to provide annotations JSON

* refactor: regenerated those files

Co-authored-by: Maximilian <mfr@nzke.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants