Skip to content

Assets can chosen from dropdown in ui#36

Open
yash0400 wants to merge 2 commits intomasterfrom
Assets-in-ui
Open

Assets can chosen from dropdown in ui#36
yash0400 wants to merge 2 commits intomasterfrom
Assets-in-ui

Conversation

@yash0400
Copy link
Copy Markdown
Contributor

@yash0400 yash0400 commented Mar 7, 2023

No description provided.

@yash0400 yash0400 requested a review from Louis-Tarvin March 7, 2023 11:21
Copy link
Copy Markdown
Contributor

@Louis-Tarvin Louis-Tarvin left a comment

Choose a reason for hiding this comment

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

It works, although I'm not sure about the location (and colour) of the add asset button. It doesn't really make sense for it to be in the right sidebar when it is a project-wide setting. Maybe it could be added as an option in the toolbar? Like edit > add asset?
IDK maybe its fine

Base automatically changed from app-ui to master March 28, 2023 10:10
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