/uf-frontend-review-20220705 Secret
Last active
July 8, 2022 12:00
Star
You must be signed in to star a gist
This file contains 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
**Content tab** | |
I agree the works should be each on a separate row instead of a multi-select. | |
One added advantage: in the future we will most likely have a way to specify what page a certain work starts at, and it will be much easier if each is in a separate row. | |
It would also be helpful if we want to show a bit more information for each work in that tab like author (+ checkbox to copy author rel from AC). | |
We can pass some initial info to the Work state perhaps? | |
Author and language for example could be copied from the UF state | |
In creating new work inline, relationship editor doesn't work. Somehow missing the entity type: | |
https://ibb.co/MGs7HM1 | |
Out of curiosity: why are Works not created immediately like authors and publishers created inline, but rather only created once we submit the form? | |
**Details tab** | |
Extra info accordion not necessary, should be directly on the page | |
**Submit tab** | |
Repeating Catquest's comment, I agree we should show a bit more of the information on each new entity. Maybe for starters we can try the AuthorAttributes, WorkAttributes, etc. components as a base, and modify their styles if needed? | |
Lines like "Authors : 0" if I didn't create any Authors don't add any information and could be confusing, they should be removed. | |
**Entity creation modal** | |
I think the accordions/collapsible sections isn't working as well as I'd hoped. If we want to keep accordions, we should change a few things: Always have the first one open at first; should probably be renamed from "basic" to "name" ? | |
Put the "details" section right under "basic", possibly also open by default. | |
Less spacing between the sections: one of the reasons we wanted to add accordions is to save some space |
Well, most of my comments already have been addressed, nicely done :)
I'm not sure if i understand this properly but authors and publisher also not created immediately...
I spazzed out, ignore me :)
How about showing entity attributes on tooltip when user hover over it?
In my opinion that defeats the idea of an overview. It's also not the most usable pattern when taking touch devices into account.
We can —and should— however do our best to make it take as little space as possible. I think we'll be able to do some css magic to help us on that front.
Another option is to add yet another accordion to expand the entity details.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Thanks for the detailed review!
I was going with adding these relationship later on work when submitting the form, since user might later choose not to copy authors from AC so i thought it would be convenient to do this way.
Also currently POST route only support entity creation so to add authors on existing work, we will need to modify route as well
my bad, now it is fixed
I'm not sure if i understand this properly but authors and publisher also not created immediately...
How about showing entity attributes on tooltip when user hover over it? this way submit tab will remain compact & clean even for large number of entities.
agree, will remove it
Yep!