Skip to content

Conversation

@toniwowtscherk
Copy link
Collaborator

🎯 Overview

This PR implements top-left alignment for all elements in the Notification component to ensure consistent visual alignment.

📋 Changes Made

Core Component Updates:

  • NotificationItem.tsx: Updated alignItems to always use 'flex-start' for top-left alignment
  • NotificationItem.styles.ts: Added alignSelf: 'flex-start' to icon styles for consistent positioning

Storybook Stories:

  • TopLeftAlignment: New story demonstrating top-left alignment behavior with various notification types
  • MultipleLineParagraph: New story showcasing notifications with multi-line blind text content

🔍 What This Fixes

  • Ensures all notification elements (icon, title, content, close button) consistently align to the top-left
  • Provides visual examples in Storybook to verify the alignment behavior
  • Improves consistency across different notification variants and content lengths

🧪 Testing

  • Added comprehensive Storybook stories to validate alignment
  • All existing functionality preserved
  • Visual verification available in Storybook

✅ Checklist

  • Component logic updated for consistent alignment
  • Styles updated to enforce top-left positioning
  • Storybook stories added for visual validation
  • No breaking changes to existing API
  • All elements (icon, title, content, close button) align consistently

renovate bot and others added 30 commits June 2, 2025 07:30
…orepo

chore(deps): update dependency @testing-library/user-event to v14.6.1
…file

chore(deps): update dependency chromatic to v11.29.0
…ockfile

fix(deps): update dependency use-debounce to v10.0.5
chore(deps): update dependency docker to v2.8.2
…ier-8.x-lockfile

chore(deps): update dependency eslint-config-prettier to v8.10.0
…t-2.x-lockfile

chore(deps): update dependency eslint-plugin-import to v2.31.0
…11y-6.x-lockfile

chore(deps): update dependency eslint-plugin-jsx-a11y to v6.10.2
…ont-need-lodash-underscore-6.x-lockfile

chore(deps): update dependency eslint-plugin-you-dont-need-lodash-underscore to v6.14.0
* fix: set SVG viewport to 24 24 to render correctly

* chore: add changeset

---------

Co-authored-by: Kathrin Holzmann <[email protected]>
* chore: add description of package

* chore: update package.json

chore: update package.json

* chore: publish new pre-release version

* chore: update package.json

* chore: remove from ignore list
* chore: update package.json to ignore i18n-utils package

* chore: ignore i18n-utils package when generatign releases
* fix(ComponentSource): code block padding

* chore: bring latest versions
missating and others added 30 commits September 4, 2025 11:04
docs(icons): add icon component name changes to the migration guide
* docs: update README.md [skip ci]

* docs: update .all-contributorsrc [skip ci]

---------

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
* feat: add `GlobeXIcon` & `TrashIcon`

* fix: revert TrashIcon change
* feat(usagecount): makes included word switcheable in UsageCount; Fix tooltip type in UsageCard

* fix: makes includedLabel string in UsageCount

* chore: adds changeset
* refactor: improve types and fix a11y issue

* test: add unit tests for Pagination

* chore: add changeset

* Update .changeset/empty-beds-cheat.md

Co-authored-by: Rémy Lenoir <[email protected]>

---------

Co-authored-by: Rémy Lenoir <[email protected]>
* feat: use `TrayArrowDown` & `TrayArrowUp`
…n (#3182)

* feat: change variants to color for icon usage

* feat: add variant to color token util func

* feat: add changeset

* feat: update badge pkg

* feat: change icon color on warning variant

* chore: check website build

* chore: re run website build
- Update NotificationItem to always use flex-start alignment
- Add alignSelf: 'flex-start' to icon styles for consistent top-left positioning
- Add TopLeftAlignment and MultipleLineParagraph stories to showcase the alignment behavior
- Ensure all notification elements (icon, title, content, close button) align to top-left
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.