a dashed outline is left on elements on secondary pages.
to reproduce
- quick edit 'confiture Site branding'
- in 'toggle branding elements'
- click either link in "Defined on the Appearance Settings or Theme Settings page."
- then, new page load under Seven theme. all elements have dashed outline on them.
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | interdiff.txt | 1.04 KB | star-szr |
| #4 | 2901205-4.patch | 3.73 KB | tedbow |
| Screen Shot 2017-08-10 at 7.23.35 AM.png | 129.17 KB | skaught | |
| Screen Shot 2017-08-10 at 7.23.25 AM.png | 94.75 KB | skaught | |
| Screen Shot 2017-08-10 at 7.23.35 AM.png | 129.17 KB | skaught |
Comments
Comment #2
skaughtComment #3
tedbow@SKAUGHT nice catch.
I think it even easier to reproduce
Pretty sure this caused it #2784853: Determine when Outside In library should be loaded: piggyback on contextual_toolbar() but not sure why it is happening because Seven removes all contextual links so we should not have the behavior.
Comment #4
tedbowOk this is caused by changes from #2784853: Determine when Outside In library should be loaded: piggyback on contextual_toolbar().
That issue simplified the module by always attaching the outside_in library if the contextual item was in the toolbar.
Then we just check
localStorage.getItem('Drupal.contextualToolbar.isViewing') === 'false'Which will be true if you are in edit mode regardless of whether there any actually contextual links on the page.
So this patch changes to only do this check when the first Contextual link is added to the page.
Comment #5
skaughtpatch does seem to correct base issue. i'm not sure what other level of review would be good at this point.
Comment #9
star-szrs/triggger/triggered/
Fixed on commit, interdiff attached.
Committed and pushed c67c0e302c to 8.5.x and 98f51ad5f1 to 8.4.x. Thanks!
Comment #10
wim leersI've reported this bug several times, glad to see it fixed!
Comment #11
tedbowChanging to new settings_tray.module component. @drpal thanks for script help! :)