The persistent chat, as mentioned in ref T15626, when collapsed, still has an invisible gear icon on the left of the upper arrow icon. It should not be there. This fix removes the core-menu-item class if the chat is minimized and add it back when expanded.
Details
Details
- Reviewers
- None
- Group Reviewers
O1: Blessed Committers - Maniphest Tasks
- T15626: [Web] Persistent chat settings should not be clickable or visible when the chat is minimized
Follow the steps in task ref: T15626 and now the context menu should not be triggered when clicking on the left of the upper arrow
Diff Detail
Diff Detail
- Repository
- rP Phorge
- Branch
- task-T15626 (branched from master)
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 820 Build 820: arc lint + arc unit
Event Timeline
Comment Actions
Check out https://we.phorge.it/book/contrib/article/javascript_coding_standards/ and the Javeline docs here: https://github.com/phacility/javelin/tree/master/src/docs/concepts
(I'll try to render the Javeline guide here later).
- We don't use document.querySelector - we use JX.DOM.find, you can see an example in line 43
- the else should be in the same line as the }.