Islandora / documentation

Contains islandora's documentation and main issue queue.
MIT License
104 stars 71 forks source link

Some UX tweaks for content managers #1186

Closed mjordan closed 5 years ago

mjordan commented 5 years ago

I'd like to suggest a couple of tweaks that I think would improve the UX for people adding and editing content. These are just suggestions, counter arguments are welcome.

Move the "System" field group in the repository item add/edit form to the top

This field group contains the only required field (besides the title) for that content type. I think it should go at the top of the form, not be burried down at the end.

When adding Media, on form submission, Drupal should redirect to the node edit form, not to the list of media.

There is no easy way back to the node from that list.

Default value of the "Media Use" list in the GUI should be "Original File".

Currently it is "None". Most times this list will be used it to upload an original file.

I know that many of these things are trivial to change locally (e.g., the location of the "System" field group) but I'm thinking of first impressions.

manez commented 5 years ago
mjordan commented 5 years ago

It's a bit awkward at the end, but I'm used to scrolling all the way down for save/publish options in Drupal, so this one hasn't bothered me.

Good point. Might be more of a personal preference, and I can see where some people might not like it at the top.

dannylamb commented 5 years ago

@manez Default value can be set on a per bundle basis. For example, for Image media: http://localhost:8000/admin/structure/media/manage/image/fields/media.image.field_media_use

You'd have to set that to Original File for all the other media types too, but it's not too onerous.

Awesome suggestions, btw @mjordan.

mjordan commented 5 years ago

Another one: an action that combines "Delete content", "Delete Node from Fedora", and "Delete Node from Triplestore". It's a PITA to have to do all three if you want to purge all aspects of an object.

dannylamb commented 5 years ago

If you just delete content, the others will trigger in response to the delete event. Or at least they should. The one I would add is 'Delete object and all media'.

On Wed, Jun 26, 2019, 00:58 Mark Jordan notifications@github.com wrote:

Another one: an action that combines "Delete content", "Delete Node from Fedora", and "Delete Node from Triplestore". It's a PITA to have to do all three if you want to purge all aspects of an object.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Islandora-CLAW/CLAW/issues/1186?email_source=notifications&email_token=AE6PSH6MVNHDFW3NZ6O7KOTP4LSOPA5CNFSM4H26M3YKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYSH4MY#issuecomment-505708083, or mute the thread https://github.com/notifications/unsubscribe-auth/AE6PSH6K4RS7YXEDXN5DU5LP4LSOPANCNFSM4H26M3YA .

manez commented 5 years ago

@dannylamb I'm clear on how to change it in the front end (Drupal Form GUI :heart_eyes:) but I'm not sure how to put it into a pull request so that it builds that way from now on

manez commented 5 years ago

After some advice from @seth-shaw-unlv on the last tech call, I think I've got the right place to make changes, and the right changes in place. Now, can anyone help me figure out how to test them? Can I point claw-playbook at my fork of islanodra_defaults to make sure the form edits show up as their supposed to? How?

seth-shaw-unlv commented 5 years ago

If you take a box that is already setup you can vagrant ssh in, change to the relevant module directory, pull in your branch’s changes, then do a feature import.

Doing it from the playbook is best done on a branch of the official repos, rather than forks.

manez commented 5 years ago

"vagrant ssh in, change to the relevant module directory, pull in your branch’s changes" this is kind of where I wander into the woods and get lost, but I'll keep at it.

Edit: Success! Shenanigans involved to use a GUI to SSH into my VM, but success!

mjordan commented 5 years ago

PR merged in fef6b30eab3b0c365e82c902122f67f2da152ca7.