Skip to content

Conversation

@Bofan-Zhu
Copy link
Contributor

@Bofan-Zhu Bofan-Zhu commented Jan 28, 2026

Checklist

General

Motivation and Context

Currently, the buttons in the header visually clash too much with buttons in the content areas. To reduce this visual noise and we are transforming the header buttons to text buttons.

Description

This PR:

  • changes the header buttons to use variant "text"
  • reduces the width of the buttons
  • adds a "divider" between the buttons to visually separate both

Steps for Testing

Prerequisites:

  1. Navigate to TumApply and view the header
  2. Check that visual noise is reduced and buttons are still visually detached from each other

Review Progress

Code Review

  • Code Review 1

Manual Tests

  • Test 1

Screenshots

Before:
image
Now:
image

@Bofan-Zhu Bofan-Zhu self-assigned this Jan 28, 2026
@Bofan-Zhu Bofan-Zhu requested a review from az108 as a code owner January 28, 2026 13:29
@Bofan-Zhu Bofan-Zhu linked an issue Jan 28, 2026 that may be closed by this pull request
2 tasks
@github-actions github-actions bot added the client Pull requests that update TypeScript code. (Added Automatically!) label Jan 28, 2026
@Bofan-Zhu Bofan-Zhu added shared Code changes in shared modules ui/ux improvement client Pull requests that update TypeScript code. (Added Automatically!) ready for review and removed client Pull requests that update TypeScript code. (Added Automatically!) labels Jan 28, 2026
Copy link
Contributor

@sachmii sachmii left a comment

Choose a reason for hiding this comment

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

Looks great, thank you!

Copy link
Contributor

@Abi107717 Abi107717 left a comment

Choose a reason for hiding this comment

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

Tested locally and Code LGTM as well, thank you 😊

@Bofan-Zhu Bofan-Zhu added ready for merge PR is ready to merge and removed ready for review labels Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

client Pull requests that update TypeScript code. (Added Automatically!) ready for merge PR is ready to merge shared Code changes in shared modules ui/ux improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adjust sign in button

4 participants