feat: resolve back button overlapping#1327
Conversation
WalkthroughThis update modifies the position of the "Back" button container on the About page, changing its alignment and offsets for different screen sizes. Additionally, it alters the version specifier for the "superjson" dependency in the package configuration to use a "catalog:" prefix. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant AboutPage
participant BackButton
User->>AboutPage: Loads About page
AboutPage->>BackButton: Renders Back button at new responsive position
User->>BackButton: Clicks Back button
BackButton->>AboutPage: Triggers navigation
Possibly related issues
Possibly related PRs
Suggested reviewers
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (1)
🔇 Additional comments (1)
✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
apps/mail/app/(full-width)/about.tsx(1 hunks)apps/mail/app/(full-width)/contributors.tsx(1 hunks)
🔇 Additional comments (1)
apps/mail/app/(full-width)/about.tsx (1)
13-19: Placement tweak looks goodSwitching to
right-4 top-6and resetting withmd:left-8 md:right-autocleanly resolves the overlap on small screens while preserving the original position on ≥ md. No functional or a11y concerns spotted.
MrgSub
left a comment
MrgSub
left a comment
There was a problem hiding this comment.
Please address coderabbit
Manav437
force-pushed
the
fix/back-button-overlap
branch
from
9c98c8a to
549be25
Compare
Manav437
left a comment
Manav437
left a comment
There was a problem hiding this comment.
Fixed the back button in apps/mail/app/(full-width)/about.tsx
|
Fixed, thank you |
2 participants
Description
This PR fixes a UI issue where the back button was overlapping with the menu button in the full-width layout of the /about page.
The button's absolute positioning caused visual conflict. The layout has been adjusted to properly space the elements.
Type of Change
Areas Affected
Testing Done
Checklist
Screenshots
Before:
After:
By submitting this pull request, I confirm that my contribution is made under the terms of the project's license.
Summary by CodeRabbit
Style
Chores