Skip to content

Conversation

thecoolwinter
Copy link
Collaborator

@thecoolwinter thecoolwinter commented Aug 28, 2025

Description

Updates the toolbar for macOS Tahoe. All changes in this PR should not affect existing functionality or design.

Related Issues

Checklist

  • Navigator extends to the top of the toolbar.
  • Task start/stop buttons in group.
  • Toolbar no longer compact (taller)
  • Extra padding on branch picker.
  • Activity viewer no longer has a background color.
  • Schema select hover is rounded.
  • Task select hover is rounded.
  • Activity viewer padding.
  • Task status padding too short.
  • Task status animation slightly wrong.
  • Notifications and activity viewer are combined in one glass item.
  • Notification item keeps space when hidden.
  • I read and understood the contributing guide as well as the code of conduct
  • The issues this PR addresses are related to each other
  • My changes generate no new warnings
  • My code builds and runs on my machine
  • My changes are all related to the related issue above
  • I documented my code

Screenshots

All toolbar sta
Screenshot 2025-08-29 at 4 33 01 PM
Screenshot 2025-08-29 at 4 32 57 PM
Screenshot 2025-08-29 at 4 32 44 PM
Screenshot 2025-08-29 at 4 32 36 PM
tes:

Compression

Screen.Recording.2025-08-29.at.4.33.20.PM.mov

Toolbar in action

Screen.Recording.2025-08-29.at.4.30.35.PM.mov

Popovers

Screen.Recording.2025-08-29.at.11.58.48.AM.mov

Task notification viewer

Screen.Recording.2025-08-29.at.12.23.59.PM.mov

@thecoolwinter thecoolwinter added enhancement New feature or request macOS Tahoe Issues for macOS Tahoe labels Aug 28, 2025
@jacob-carlborg
Copy link

Looking at the images, the bottom line of the toolbar is not aligned with the line in the sidebar and in the inspector. It's not a straight line across the toolbar and the sidebars. Is that something that Apple's frameworks are rendering or CodeEdit?

@SilverMarcs
Copy link

It seems the toolbar has significantly longer height than it should have

@thecoolwinter
Copy link
Collaborator Author

Looking at the images, the bottom line of the toolbar is not aligned with the line in the sidebar and in the inspector. It's not a straight line across the toolbar and the sidebars. Is that something that Apple's frameworks are rendering or CodeEdit?

It's a bit of both. The built-in toolbar no longer has a set size across each so we'll probably follow Xcode's lead and change it up. That's not actually a part of the toolbar in CE though, so it'll be a different PR.

@thecoolwinter
Copy link
Collaborator Author

It seems the toolbar has significantly longer height than it should have

It's larger now but it also looks significantly larger because we have some hard-coded top padding in components that will need to be updated.

@SilverMarcs
Copy link

It seems the toolbar has significantly longer height than it should have

It's larger now but it also looks significantly larger because we have some hard-coded top padding in components that will need to be updated.

May I ask which components need this top padding removed?

@thecoolwinter
Copy link
Collaborator Author

It seems the toolbar has significantly longer height than it should have

It's larger now but it also looks significantly larger because we have some hard-coded top padding in components that will need to be updated.

May I ask which components need this top padding removed?

It should be the EditorLayoutView iirc and I think there's some in the navigator code. tbh I haven't looked into it yet.

@thecoolwinter
Copy link
Collaborator Author

Ah shoot, you're totally right it is the toolbar making that space...

Screenshot 2025-08-29 at 12 44 19 PM

@thecoolwinter
Copy link
Collaborator Author

Good catch, figured it out. Can't use the .unified toolbar style it has to be .automatic.

@thecoolwinter
Copy link
Collaborator Author

Well that unified to default change led to a whole slew of bugs but I think this is ready to go!

@thecoolwinter thecoolwinter marked this pull request as ready for review August 29, 2025 21:37
@SilverMarcs
Copy link

Yeah size seems perfect now. But the toolbar background is getting this transparent effect from the tahoe default toolbar styling.
I'd assume .scrollEdgeEffectStyle(.hard) could help but couldn't get that to work

2025-08-30_07-12-29

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request macOS Tahoe Issues for macOS Tahoe
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants