Skip to content

Add shadow to sticky headers to improve visual clarity#404

Closed
BenjaminSchaaf wants to merge 1 commit into
packagecontrol:gh-pagesfrom
sublimehq:sticky-header-shadows
Closed

Add shadow to sticky headers to improve visual clarity#404
BenjaminSchaaf wants to merge 1 commit into
packagecontrol:gh-pagesfrom
sublimehq:sticky-header-shadows

Conversation

@BenjaminSchaaf

Copy link
Copy Markdown

No description provided.

@kaste

kaste commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

Looks good. braver did no shadows at all I think, but to me it looks good. There is minor lint and at most one could say

image

that the div is looking strange, and an attribute or class-name saying, "hej, I'm a scroll position sentinel. I have no content" would be ok.

@BenjaminSchaaf BenjaminSchaaf force-pushed the sticky-header-shadows branch from 5af5b0c to 3fcf7f0 Compare June 16, 2026 02:44
@BenjaminSchaaf

Copy link
Copy Markdown
Author

I've pushed those changes

@kaste

kaste commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Easier when you tick the "allow maintainers to edit"

image

Please rebase.

@kaste

kaste commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

I just pick the one commit directly. 👍

@kaste kaste closed this Jun 16, 2026
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.

2 participants