[alchemist] improve UI#151
Draft
StupidRepo wants to merge 10 commits into
Draft
Conversation
- added really nice indicator for removing items - fixed a warning
- add custom filtering by first letter in name - TODO: fix keyboard gap
Author
|
if i remove scaffolds everywhere (or ignore their padding value, at the very least), this still seems to happen? |
Author
Detailsscreen-20260522-203445-1779478453014.mp4(outdated, see comment below) demo |
- fix item list scroll position resetting when fully faded out
Author
another, updated demo (showing commit 0d24cea)Detailsscreen-20260522-224319-1779486142967.mp4
improved item details screen (showing commit 1412564) |
…fications. - added better fallback icon, and added undiscovered icon variant. - add star icon for final items to make them easier to identify. - item details screen shows recipes that make the item, and recipes the item is used in (but only shows these recipes when all output items for that recipe are unlocked)
8dbd2ec to
80db339
Compare
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.


marked as draft due to bug. see below.
this PR also updates
library/util/Navigation.ktdue to a weird bug with IME padding, hence why the PR is a draft.i have tried messing around with removing
.imePadding()from the modifiers list, as well as attempting to consume it before getting to alchemist'sHomeScreen.kt, but no matter what I do it either results in theHomeScreen.kt's first box (in the scaffold) either not moving up enough or moving up too much.the red background is applied to alchemist's
HomeScreen.kt's first box, as a way of demonstrasting what is happening.edit: still can't fix this bug - this is why i hate compose so much 🥀