This issue has been spun off from https://www.drupal.org/project/easy_breadcrumb/issues/2912889, as an attempt to make the admin interface for easy breadcrumb easier to understand.
Below is a list of changes that will make the user interface less overwhelming to users, and easier to understand.
* Replace the word "segments" with "crumbs"
* Update grammar in checkbox labels.
* Remove description text when it's nearly identical to checkbox labels.
* Use #states on dependent form elements.
* Use placeholders instead of examples in description text, where useful.
* Use real world examples where sensible.
* Organize into two sections; 1) Breadcrumb text 2) Breadcrumb capitalization
Other changes in this patch:
* Enclosing strings with ' ' instead of " " is a micro-optimization for performance, as PHP will not try to substitute variables in strings enclosed with ' '.
* Fixing minor code style updates (fixed indentation, added trailing commas)
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | easy_breadcrumb-admin_cleanup-2938421-8.patch | 18.79 KB | tatarbj |
Comments
Comment #2
jenlamptonHere's a patch for review.
Comment #3
tatarbjI've just renamed the issue as it's not really a rework, also i feel even there is a patch that i have to admit works well and improves a lot on the interfaces, i would go a bit further and try to push some other improvements for mainly the administration interfaces. The ones that are included in the patch can be considered the first steps, let me get back in a few days to showcase what else we could do for the next version of easy_breadcrumb.
I've also changed the category to 'plan', all contributions and suggestions are warmly welcome by other folks!
Bests,
Balazs.
Comment #4
tatarbjComment #5
jenlamptonThanks @tatarbj! I've rerolled this patch with a few other minor improvements including adjustments to the text field sizes and better labels in some places. This matches what was committed in the Backdrop version of the module. I'd love to see what other improvements you have in mind, and incorporate them for Backdrop as well.
Comment #6
tatarbjI attach an interdiff that shows the changes between the two patches
Comment #7
tatarbjFor now i don't find other things to improve on the d7 version admin interface, so to push it forward, i'm committing the second patch as rtbc. Thanks @jenlampton for the idea and implementation too!
Comment #9
tatarbjjust a small reroll had to happen as a few coding standards issues were already committed and somehow git apply didn't like the patch for the first, so i upload the one that's gonna be committed now.
Comment #11
tatarbjComment #12
jenlampton