Skip to content

helium/ui/layout: remove margin after pinned in vertical layout#1513

Open
Jasdeep-Dhillon wants to merge 1 commit intoimputnet:mainfrom
Jasdeep-Dhillon:remove-margin-after-pinned
Open

helium/ui/layout: remove margin after pinned in vertical layout#1513
Jasdeep-Dhillon wants to merge 1 commit intoimputnet:mainfrom
Jasdeep-Dhillon:remove-margin-after-pinned

Conversation

@Jasdeep-Dhillon
Copy link
Copy Markdown

For your pull request to not get closed without review, please confirm that:

  • An issue exists where the maintainers agreed that this should be implemented
    (an approved feature request, or confirmed bug).
  • I tested that my contribution works locally, and does not break anything,
    otherwise I have marked my PR as draft.
  • If my contribution is non-trivial, I did not use AI to write most of it.
  • I understand that I will be permanently banned from interacting with this
    organization if I lied by checking any of these checkboxes.

Tested on (check one or more):

  • Windows
  • macOS
  • Linux

Related to issue #1498 which mentioned to remove the extra margin after pinned tabs when using vertical layout. I have removed the check which used collapsed state to show the separator which resulted in extra margin after pinned tab.

Before:
image

After:
image

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Apr 29, 2026

Reviews (1): Last reviewed commit: "helium/ui/layout: remove margin after pi..." | Re-trigger Greptile

Comment thread patches/helium/ui/layout/remove-margin-vertical-pinned-tabs.patch Outdated
@Jasdeep-Dhillon Jasdeep-Dhillon force-pushed the remove-margin-after-pinned branch from 15c316e to 59a14af Compare April 29, 2026 05:51
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.

1 participant