Skip to content

Wrong base branch to fix CSS#457

Merged
ashleywolf merged 1 commit intomainfrom
maintainer-in-academia-css
May 5, 2026
Merged

Wrong base branch to fix CSS#457
ashleywolf merged 1 commit intomainfrom
maintainer-in-academia-css

Conversation

@samus-aran
Copy link
Copy Markdown
Collaborator

Whoops I missed this - Wrong base branch to fix CSS

#456

Screenshot 2026-05-05 at 20 30 26 Screenshot 2026-05-05 at 20 30 39

Agent-Logs-Url: https://github.com/github/maintainermonth/sessions/03782496-2df7-4082-a1f4-9ba5b14ccd20

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: samus-aran <4041275+samus-aran@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 5, 2026 19:36
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

This PR corrects a CSS fix that was previously opened against the wrong base branch. In the academia listing UI, it aligns .academia-list__main content to the left at all viewport sizes so the layout matches the intended presentation from PR #456.

Changes:

  • Removes the default text-align: right from .academia-list__main.
  • Removes the now-unnecessary $xl media-query override.
  • Leaves the academia list with a single unconditional text-align: left rule.
Show a summary per file
File Description
components/academia/academia.scss Simplifies the academia list text alignment rule so content is left-aligned consistently across breakpoints.

Copilot's findings

Tip

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

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@ashleywolf ashleywolf merged commit 9d943bd into main May 5, 2026
8 checks passed
@samus-aran samus-aran deleted the maintainer-in-academia-css branch May 5, 2026 23:08
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.

4 participants