Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enhance Shop/Upgrade tree UI #243

Open
Monkey-bored opened this issue Mar 1, 2024 · 5 comments
Open

Enhance Shop/Upgrade tree UI #243

Monkey-bored opened this issue Mar 1, 2024 · 5 comments
Assignees
@Monkey-bored
Labels
UI Change in the UI
Milestone
0.2.0

Comments

@Monkey-bored
Copy link
Collaborator

Monkey-bored commented Mar 1, 2024 •
edited by Ido-Barnea
Loading

Requested Changes

Currently the shop/upgrades tree items just seem to float somewhere, instead add a visual border around the the shop/upgrades tree and text at the top of the area inside the border that says "Shop" or "Upgrades tree" accordingly. See the sketch below:

image
@Monkey-bored Monkey-bored added the UI Change in the UI label Mar 1, 2024
@Monkey-bored Monkey-bored self-assigned this Mar 1, 2024
@Monkey-bored Monkey-bored added this to the 0.2.0 milestone Mar 1, 2024
@Ido-Barnea
Copy link
Owner

@Monkey-bored May I also suggest this change?
image

@Monkey-bored
Copy link
Collaborator Author

@Monkey-bored May I also suggest this change?
image

It kind of hides the items svg behind the price

@Ido-Barnea
Copy link
Owner

@Monkey-bored May I also suggest this change?
image

It kind of hides the items svg behind the price

Yes well I'm not an MS Paint God or anything but my intention was that the items would be higher (guess I could have painted it correctly but 🤓)

@Ido-Barnea
Copy link
Owner

@Monkey-bored May I also suggest this change? image

@Monkey-bored Please only add the border and title as my previous suggestion will be solved in #250

@Ido-Barnea
Copy link
Owner

Ido-Barnea commented May 3, 2024 •
edited
Loading

@Monkey-bored could you please update your progress regarding this issue?

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

@Monkey-bored Monkey-bored

Labels
UI Change in the UI
Projects
None yet
Milestone
0.2.0
Development

No branches or pull requests
2 participants
@Ido-Barnea @Monkey-bored