[wp-trac] [WordPress Trac] #34923: Introduce basic content authorship in the Customizer
WordPress Trac
noreply at wordpress.org
Fri May 27 02:44:43 UTC 2016
#34923: Introduce basic content authorship in the Customizer
-------------------------------------+-------------------------------------
Reporter: westonruter | Owner: westonruter
Type: enhancement | Status: accepted
Priority: normal | Milestone: 4.6
Component: Customize | Version:
Severity: normal | Resolution:
Keywords: needs-patch has-patch | Focuses: ui, accessibility,
ux-feedback ui-feedback needs- | javascript
testing |
-------------------------------------+-------------------------------------
Comment (by celloexpressions):
Note following up on the discussion in #design,
[attachment:34923.ui.1.diff] can and should be tested for the UI (without
needing the plugin), but don't hit save & publish as the menu items aren't
real. Delete them before saving.
@westonruter I don't think a post type parameter would be appropriate here
because it's basically an extention of `show_in_nav_menus`, with a
specific part of that UI. Generically showing it in the customizer in the
context of customize-posts would mean something different and shouldn't
necessarily be tied to the menus interface as well (in addition to not
being appropriate for core yet; this would presumably be more related to
being able to edit content in the customizer I think, and should be saved
for that future use). So I think the best behavior would be for it to be
on by default, but able to be disabled with a filter so that core types
could also be disabled if desired.
@mapk (and everyone who participated in the design chat): the placement
when adding to the list of menu items will be tricky. However, if it's
added to the bottom of the list it will ''always'' trigger an infinite
scroll, for each item added, and as a result the new items wouldn't be
next to each other. The best option is probably to insert them at the top
so that they're all together. Since this would cause the list to jump down
anyway, it should probably auto-scroll to the top when this happens (but
that would only happen once if multiple consecutive items are added).
Since this is designed to be most useful during the site setup context,
this decision won't ultimately matter too much since these would be the
first items in the list.
Regarding a separate step to add to menu, I think that would be excessive,
similar to [comment:13]. Since the primary flow here is to create items so
that they can be added to the menu, auto-adding to both places is probably
the most appropriate action. If it were a two-step process this flow would
become fairly cumbersome, especially for keyboard-only users, but also
with touch, especially when adding multiple items at a time. Since we're
targeting the site-setup flow primarily, auto-add to the menu is probably
the best approach, and there is an existing workflow to remove (and bulk-
remove) items from there if a user changes their mind. I would also guess
that new users would have trouble understanding the additional step of
adding to a menu, based on previous user testing with the old and new
menus interfaces (we should user-test that).
@afercia My thought in terms of accessibility would be to move it to the
top of the list in tab order but keep it at the bottom visually. Infinite
scroll is definitely an accessibility challenge here; pretty much the only
way to sanely navigate the available menu items panel is to tab back up to
collapse the section before moving to the next one, and putting the input
at the top in terms of the html structure would make that easier from
there. Perhaps a bit disorienting to jump like that, but probably better
than the alternative.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/34923#comment:43>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list