Skip to content

Conversation

@huuaho
Copy link
Contributor

@huuaho huuaho commented Dec 29, 2025

C3DC-2003

Please Approve the PR on the library side before this PR
CBIIT/bento-frontend#1181

Design QA updates

220 - #220 Text should be uppercase, not capitalized; Also make sure size is 17 pt and line spacing at 16

221 - #221 There should be 0px of space between tabs and buttons

263 - Tab color borders are all the same when activated: #006A8F

265 -

  1. Dropdown should be 3 px from More button
  2. Confirm font size of both button text and dropdown text
  3. Decrease spacing between each line item in dropdown
  • 221 and 263 were already resolved

  • 265 is fixed in the package update as well as an updated styling for the manage columns popup divider.

Copilot AI review requested due to automatic review settings December 29, 2025 19:24
@huuaho huuaho added the 🚧 Do Not Merge Work under process label Dec 29, 2025
Copy link
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 implements Design QA updates for the explore tabs and table interface, focusing on text styling, spacing adjustments, and dropdown menu refinements. The changes address specific Figma design specifications to improve visual consistency and alignment with design standards.

Key changes:

  • Updated text transform from capitalize to uppercase for tab labels with adjusted font size and line spacing
  • Removed spacing between tabs and buttons
  • Adjusted dropdown menu item padding and spacing for improved visual hierarchy

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

@huuaho huuaho removed the 🚧 Do Not Merge Work under process label Dec 29, 2025
@huuaho huuaho requested a review from amattu2 December 29, 2025 20:10
@amattu2
Copy link
Contributor

amattu2 commented Dec 30, 2025

@huuaho Can you resolve the merge conflicts on this?

amattu2
amattu2 previously approved these changes Dec 30, 2025
Copy link
Contributor

@amattu2 amattu2 left a comment

Choose a reason for hiding this comment

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

LGTM. Once you resolve the merge conflicts, you can merge this.

@amattu2 amattu2 added this to the 1.8.0 milestone Dec 30, 2025
@amattu2 amattu2 merged commit 7fbec30 into 1.8.0 Dec 30, 2025
4 checks passed
@amattu2 amattu2 deleted the C3DC-2003 branch December 30, 2025 16:10
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.

3 participants