Repository navigation
Even out the commit action row, full-width hairlines, fixed logo size - #80
Open
shanerbaner82 wants to merge 2 commits into
Open
shanerbaner82 wants to merge 2 commits into
shanerbaner82 wants to merge 2 commits into
Conversation
…the logo size The star, pastel, Quote and Reply actions now share the row equally, X-style, with Reply as an icon like the rest. The hairline under a commit takes the rail's gutter as padding, and under a thread it's the last reply's own border, so both run the row's full width. The nav bar logo gets an explicit 105x28 size, as it pulsed in and out on device without one. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…n it Share was the first item of More. It's now a button of its own at the end of the row, like X's, and More keeps Pin, Report and Delete, and isn't drawn when there are none, such as on someone else's commit that isn't approved yet. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Verified
Pest passes (1188 tests, arch test excluded: it runs out of PHP's default memory). Checked on an iPhone 15 Pro Max from a screen recording; the action row and hairline changes looked right, and the logo fix was confirmed there too.
Not in this PR
App Store Connect lists the in-flight version as 1.0 with bundle ID
com.nativephp.devnull. Those are set on the Bifrost side, not in the repo.🤖 Generated with Claude Code