Skip to content

egui: continue visual refinements#3980

Merged
Parth merged 3 commits intomasterfrom
egui-preview-pending-share
Nov 25, 2025
Merged

egui: continue visual refinements#3980
Parth merged 3 commits intomasterfrom
egui-preview-pending-share

Conversation

@Parth
Copy link
Copy Markdown
Member

@Parth Parth commented Nov 25, 2025

@ad-tra refined the UI in #3948 which was well received by all.

However due to all the copy and pasting we're doing in the filetree.rs implementation, the visual changes didn't apply to suggested docs. If you look in that screenshot it looks like adam's folder is a parent of suggested docs. Suggested docs' icon also doesn't open and close. Also the documents inside suggested docs aren't aligned nicely, and they have extensions when they shouldn't.

This PR consolidates the code that draws a file cell so that it can be drawn with a uniform appearance.

I need a reliable visual abstraction for showing file tree like things for #3511

before:
image
image

after:
image
image

@Parth Parth changed the title egui: revamp pending share experience egui: continue visual refinements Nov 25, 2025
@Parth Parth marked this pull request as ready for review November 25, 2025 22:41
@Parth Parth merged commit ee6b262 into master Nov 25, 2025
1 check passed
@Parth Parth deleted the egui-preview-pending-share branch November 25, 2025 22:41
image = "0.24"
lb-fonts = { git = "https://github.com/lockbook/lb-fonts" }
hayro = "0.4.0"
hayro = "0.3.0"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Parth was this intentional?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I fixed in a subsequent commit a couple minutes later

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants