Infrastructure/ghost
0
Fork 0
mirror of https://github.com/TryGhost/Ghost.git synced 2025-02-24 23:48:13 -05:00
Code Issues Activity

🐛 Fixed tag/author removal in post settings removing multiple tags/authors

Browse source 
closes https://github.com/TryGhost/Ghost/issues/9575
- in 6a3cfc2ca8 an `onmouseover` event handler was added for the close button in token input options which resulted in double firing in some circumstances where our code fired then the ember-power-select removal event handler also fired
This commit is contained in:
Kevin Ansfield 2018-04-23 14:25:59 +01:00
parent f11cb2e491
commit 5de0ae5cea
2 changed files with 5 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
ghost/admin/app/components/gh-token-input/trigger.js
View file

@ -12,10 +12,12 @@ export default EmberPowerSelectMultipleTrigger.extend({
}, },
handleOptionMouseDown(event) { handleOptionMouseDown(event) {
if (!event.target.closest('[data-selected-index]')) {
let action = this.get('extra.optionMouseDown'); let action = this.get('extra.optionMouseDown');
if (action) { if (action) {
return action(event); return action(event);
} }
}
}, },
handleOptionTouchStart(event) { handleOptionTouchStart(event) {

1
ghost/admin/app/templates/components/gh-token-input/trigger.hbs
View file

@ -27,7 +27,6 @@
aria-label="remove element" aria-label="remove element"
class="ember-power-select-multiple-remove-btn" class="ember-power-select-multiple-remove-btn"
data-selected-index={{idx}} data-selected-index={{idx}}
onmousedown={{action "chooseOption" opt}}
> >
{{svg-jar "close" data-selected-index=idx}} {{svg-jar "close" data-selected-index=idx}}
</span> </span>