[wp-trac] [WordPress Trac] #37974: Add multi-panel feature to pages through add_theme_support
WordPress Trac
noreply at wordpress.org
Wed Oct 12 23:49:32 UTC 2016
#37974: Add multi-panel feature to pages through add_theme_support
-----------------------------------------+------------------
Reporter: karmatosed | Owner:
Type: task (blessed) | Status: new
Priority: normal | Milestone: 4.7
Component: Themes | Version:
Severity: normal | Resolution:
Keywords: has-ux-feedback needs-patch | Focuses: ui
-----------------------------------------+------------------
Comment (by bradyvercher):
I talked with @davidakennedy about [comment:119] and [comment:120] and
those were tested against [attachment:37974.8.diff]. Those particular bugs
should already be fixed in [attachment:37974.9.diff].
Replying to [comment:123 @helen]:
> The usage of the post ID in the fragment URL is a return to ugly
permalinks - I'm not super excited about using # at all, but whatever it
is, it should stick to the slug (which should still be editable).
I can update those to use the slug, but that leaves a few things that need
to be worked out:
1. Is there an alternative to using the hash?
2. Should we update the separator that's used for page hierarchy? For
example, the page URI for a child would currently be `parent-page/child-
page`. Do we replace the slash with a dash or underscore instead? The HTML
4 spec doesn't appear to allow slashes in the `id` attribute, while HTML 5
does.
3. If the separator is updated, to what extent should we go to ensure the
fragments are unique to prevent collisions?
4. How should themes ensure the anchor is present on the page? A new
template tag?
> What are plans for indicating that X pages are the fragments that make
up Y page on the pages list table?
I'm all ears if anyone has any ideas on this front.
> Does this have to hinge on theme support?
I think it would be a poor experience for users to try to use the feature
in a theme that doesn't support it. At the very least, themes need to make
sure the anchor for each section matches the fragment core generates.
Themes may also need to register a container selector for selective
refresh to work unless we can inject a temporary container in the
Customizer preview using `loop_start` and `loop_end`.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/37974#comment:125>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list