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

[Notification-overlay] Title classes get margin-top on modal with internet styles #1931

Closed
imagoiq opened this issue Sep 7, 2023 · 2 comments · Fixed by #2407
Closed

[Notification-overlay] Title classes get margin-top on modal with internet styles #1931

imagoiq opened this issue Sep 7, 2023 · 2 comments · Fixed by #2407
Assignees
Labels
🐞 bug Something isn't working 📦 styles Related to the @swisspost/design-system-styles package

Comments

@imagoiq
Copy link
Contributor

imagoiq commented Sep 7, 2023

When using @use "@swisspost/design-system-styles/index.scss"; and code example from https://design-system.post.ch/#/post-samples/notification-overlay , we get unwanted margin-top with all titles:

image

@oliverschuerch oliverschuerch added 🐞 bug Something isn't working 📦 styles Related to the @swisspost/design-system-styles package labels Sep 13, 2023
@oliverschuerch oliverschuerch added this to the Maintenance milestone Sep 13, 2023
@imagoiq
Copy link
Contributor Author

imagoiq commented Oct 4, 2023

So the issue seems present since the beginning, here is the culprit and PR:

margin-top: map.get(spacing.$post-sizes, 'bigger-big');

This type for the portal style is only loaded on external apps (and so index.scss). Source:

I guess this was never seen as the Notification overlays are not used on external apps? I'm not sure what is the right fix. I don't think those portal´styles should be part of the design system. @gfellerph @alizedebray @oliverschuerch what do you think?

@imagoiq imagoiq modified the milestones: Maintenance, November 2023 Oct 17, 2023
@b1aserlu b1aserlu self-assigned this Dec 14, 2023
@gfellerph gfellerph removed this from the November 2023 milestone Jan 12, 2024
@b1aserlu b1aserlu assigned gfellerph and unassigned b1aserlu Jan 23, 2024
@gfellerph
Copy link
Member

@imagoiq I would prefer a temporary fix in the portal styles. With the next major version, typographic styles will get a complete overhaul and this theme will be refactored anyways. We're also looking at design tokens which will render these styles obsolete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug Something isn't working 📦 styles Related to the @swisspost/design-system-styles package
Projects
Development

Successfully merging a pull request may close this issue.

4 participants