Skip to content

docs: document notification placement - #3062

Merged
madcodelife merged 1 commit into
longbridge:mainfrom
Muhammad-Owais-Warsi:docs/notification-placement
Sep 14, 2026
Merged

madcodelife merged 1 commit into
longbridge:mainfrom
Muhammad-Owais-Warsi:docs/notification-placement

Conversation

@Muhammad-Owais-Warsi

Copy link
Copy Markdown
Contributor

Closes #3061

Description

Documents notification placement, which existed in code but not in docs:

  • global default via Theme::global_mut(cx).notification.placement (default Anchor::TopRight)
  • per-notification override via Notification::placement(Anchor) with all 8 Anchor values
    EN + zh-CN kept in sync. Docs only, no behavior change.

Screenshot

None

Break Changes

None

How to Test

  • cargo fmt --check (passed)
  • typos not available locally; CI will cover it
  • Preview website/component/notification.md and website/zh-CN/component/notification.md

Checklist

  • I have read the CONTRIBUTING document and followed the guidelines.
  • Reviewed the changes in this PR and confirmed AI generated code (If any) is accurate.
  • Passed cargo run for story tests related to the changes.
  • Tested macOS, Windows and Linux platforms performance (if the change is platform-specific)

@madcodelife

Copy link
Copy Markdown
Member

Thanks.

@madcodelife
madcodelife enabled auto-merge (squash) September 14, 2026 08:28
@madcodelife
madcodelife merged commit cc1532f into longbridge:main Sep 14, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: notification docs missing placement

2 participants