[wp-trac] [WordPress Trac] #29980: Twenty Fifteen: hidden header text control doesn't update with color scheme
WordPress Trac
noreply at wordpress.org
Fri Oct 24 01:46:44 UTC 2014
#29980: Twenty Fifteen: hidden header text control doesn't update with color scheme
---------------------------+--------------------
Reporter: cainm | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 4.1
Component: Bundled Theme | Version: trunk
Severity: normal | Resolution:
Keywords: needs-patch | Focuses:
---------------------------+--------------------
Comment (by downstairsdev):
@cainm You got closer to figuring it out than I did.
I did see an issue with this patch:
1. Start with default color scheme and title visible
2. Hide title text
3. Switch color scheme
4. Show title
5. Attempt to hide title text again (it fails to hide)
I made a small fix to resolve that issue in 29980.3.diff.
I agree this doesn't feel particularly elegant. I tried and approach using
data attributes rather than passing "changedWhileHidden" around, but that
seemed even worse.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/29980#comment:20>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list