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.

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

Some minor UX changes in external-apps #1339

Open
Yshayy opened this issue Sep 23, 2020 · 0 comments
Open

Some minor UX changes in external-apps #1339

Yshayy opened this issue Sep 23, 2020 · 0 comments

Comments

@Yshayy
Copy link
Contributor

Yshayy commented Sep 23, 2020

Related to #1332 (@AleF83), really cool we have UI for external apps, yet I think we can make it much more usable and clear with a few minor UI tweaks.
Some issues I saw:
The details label is not aligned to secrets label.
The permissions and versions are not relevant (also for creating/editing new apps) and confusing as it won't work.
The App Id should be visible as it's not a secret.

I'd suggested the following changes:
Change the details label to actual app name, and support inline editing (as in key editing). (it'll allow to remove the inner edit page)
Add Id where currently the app name resides.
Remove permissions field in set its data to empty array. (as no app should call directly to authoring, all pass gateway)
Remove version field, it's the type of the manifest, not interesting as well.
Replace buttons for secrets with small +x, the buttons are way too big.

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

No branches or pull requests

1 participant