Skip to content

Fix: Improve location text alignment on profile page#465

Open
AyushAgrawal18 wants to merge 2 commits intoNextCommunity:mainfrom
AyushAgrawal18:main
Open

Fix: Improve location text alignment on profile page#465
AyushAgrawal18 wants to merge 2 commits intoNextCommunity:mainfrom
AyushAgrawal18:main

Conversation

@AyushAgrawal18
Copy link
Copy Markdown

Fix

Improved alignment of location text on profile pages.

Changes

  • Updated layout to ensure proper right alignment
  • Fixed wrapping issue for long location names

Testing

  • Verified on /users pages
  • Checked with long location values

@AyushAgrawal18
Copy link
Copy Markdown
Author

Hi, I’ve submitted a PR for this issue. Please review.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request modifies the layout of the location section in the bio component, changing it from left-aligned to right-aligned and reducing the gap between the icon and text. The review feedback points out that these changes introduce visual inconsistencies with the rest of the sidebar and suggests using 'items-start' with the original gap size to better handle text wrapping while maintaining a uniform design.

Comment thread src/_includes/bio.njk
<div class="space-y-4 text-sm border-t border-[var(--border-color)] pt-6">
{% if location or country %}
<div class="flex items-center gap-3">
<div class="flex justify-end gap-2 text-right">
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The change to justify-end and text-right introduces a layout inconsistency with the Email and Website sections (lines 33 and 41), which are left-aligned. Additionally, the reduction of gap-3 to gap-2 breaks the uniform spacing used throughout the sidebar.

To address the wrapping issue mentioned in the PR description while maintaining UI consistency, it is recommended to use items-start instead of items-center. This ensures the icon remains aligned with the first line of text when the location name wraps, without breaking the left-aligned vertical rhythm of the profile card.

            <div class="flex items-start gap-3">

@jbampton jbampton requested a review from Copilot May 2, 2026 18:15
@jbampton jbampton self-assigned this May 2, 2026
@jbampton jbampton added this to the Turbo milestone May 2, 2026
@jbampton jbampton added this to Next May 2, 2026
@github-project-automation github-project-automation Bot moved this to In progress in Next May 2, 2026
@jbampton jbampton moved this from In progress to Review in progress in Next May 2, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Improves the visual alignment of the location/country line on user profile pages by updating the layout classes in the profile bio include.

Changes:

  • Adjusted the location row to right-align its content (justify-end, text-right) and tweaked spacing (gap-2).
  • Updated the flex row styling to better handle alignment when location text is long.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/_includes/bio.njk
Comment on lines +26 to 28
<div class="flex justify-end gap-2 text-right">
<span class="text-lg">📍</span>
<span class="text-[var(--text-muted)]">{{ location }}{% if location and country %}, {% endif %}{{ country }}</span>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Review in progress

Development

Successfully merging this pull request may close these issues.

3 participants