#30553 closed defect (bug) (fixed)
Use Customizer deep-links inside the theme modal
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 4.1 | Priority: | normal |
| Severity: | normal | Version: | 4.1 |
| Component: | Customize | Keywords: | has-patch |
| Focuses: | Cc: |
Attachments (3)
Change History (11)
#2
follow-up:
↓ 3
@
10 years ago
This'll also need to be updated in the Underscore JS template further down in that file. I really don't think we need to deep-link these here, as they extra links add a lot of clutter, and Customize is the primary action. Can we add .hide-if-customize to the existing links, but not change the URLs? That way they're only there if we have to link to the old pages and there is no Customize link, and we can reduce the clutter if they can Customize.
#3
in reply to:
↑ 2
@
10 years ago
Replying to celloexpressions:
This'll also need to be updated in the Underscore JS template further down in that file. I really don't think we need to deep-link these here, as they extra links add a lot of clutter, and Customize is the primary action. Can we add
.hide-if-customizeto the existing links, but not change the URLs? That way they're only there if we have to link to the old pages and there is no Customize link, and we can reduce the clutter if they can Customize.
Two issues here:
- Not deep-linking creates two clicks instead of one. If we really are "replacing" those header and background screens with controls in the Customizer then you need to account for that from all angles
- We need to avoid making an assumption that people will just realize that those options are now available in the Customizer. They won't.
#4
@
10 years ago
In this context, though, I think we can make those assumptions (personal opinion). But if others disagree we would just do the extra links with the alternating hide if/no customize classes. It's important that the old links remain for users that can't access the Customizer.
#5
@
10 years ago
30553.2.diff needs some testing. Not sure if the $class stuff is even needed.
#6
@
10 years ago
30553.2.diff seems to work as expected. Not even sure what the significance of the $class stuff was prior to this change, for what that's worth.
Added 30553.diff for deep-linking the Header and Background buttons. It ain't pretty but it should maintain the status quo as far as that science experiment of a code block goes :-)