mirror of
https://we.phorge.it/source/phorge.git
synced 2024-11-25 08:12:40 +01:00
Persistent Chat: fix de-minimize misclick
Summary: When the persistent chat is minimized to the bottom of your window, as mentioned in T15626, the chat bar still has an invisible square area that does nothing if you click it by mistake: {F2184778} That area was reserved for the Settings Icon, that appears only if you de-minimize the chat. The Settings Icon was toggled incorrectly from CSS. It was just hiding the Font Awesome icon, (.fa-gear), instead of hiding its clickable container. After this change, chat can be de-minimized even if you click in that specific evil spot. Original credit to @roberto.urbani for the original troubleshooting and original patch. Follow-up from: D25428 Closes T15626 Test Plan: Show the Chat, then: - when the chat is de-collapsed: - you see the Settings icon (as before) - you can click on the Settings Icon (as before) - it shows the Settings menu (as before) - when the chat is collapsed: - you can click in whatever point, and it de-collapse (better than before) Additionally, use the HTML inspector in your browser (`F12` from Firefox), to double-check that now that `<li>` has display: none when the chat is collapsed. Reviewers: O1 Blessed Committers, aklapper Reviewed By: O1 Blessed Committers, aklapper Subscribers: aklapper, tobiaswiese, Matthew, Cigaryno, roberto.urbani Maniphest Tasks: T15626 Differential Revision: https://we.phorge.it/D25628
This commit is contained in:
parent
aad704d0fc
commit
5b90567f64
2 changed files with 4 additions and 4 deletions
|
@ -9,7 +9,7 @@ return array(
|
||||||
'names' => array(
|
'names' => array(
|
||||||
'conpherence.pkg.css' => '2f25eb4f',
|
'conpherence.pkg.css' => '2f25eb4f',
|
||||||
'conpherence.pkg.js' => '020aebcf',
|
'conpherence.pkg.js' => '020aebcf',
|
||||||
'core.pkg.css' => 'cf2d74fd',
|
'core.pkg.css' => '3471f5d3',
|
||||||
'core.pkg.js' => '2eeda9e0',
|
'core.pkg.js' => '2eeda9e0',
|
||||||
'dark-console.pkg.js' => '187792c2',
|
'dark-console.pkg.js' => '187792c2',
|
||||||
'differential.pkg.css' => '6d3700f0',
|
'differential.pkg.css' => '6d3700f0',
|
||||||
|
@ -49,7 +49,7 @@ return array(
|
||||||
'rsrc/css/application/config/setup-issue.css' => '93231115',
|
'rsrc/css/application/config/setup-issue.css' => '93231115',
|
||||||
'rsrc/css/application/config/unhandled-exception.css' => '9ecfc00d',
|
'rsrc/css/application/config/unhandled-exception.css' => '9ecfc00d',
|
||||||
'rsrc/css/application/conpherence/color.css' => 'b17746b0',
|
'rsrc/css/application/conpherence/color.css' => 'b17746b0',
|
||||||
'rsrc/css/application/conpherence/durable-column.css' => '2d57072b',
|
'rsrc/css/application/conpherence/durable-column.css' => 'f68f35e3',
|
||||||
'rsrc/css/application/conpherence/header-pane.css' => 'c9a3db8e',
|
'rsrc/css/application/conpherence/header-pane.css' => 'c9a3db8e',
|
||||||
'rsrc/css/application/conpherence/menu.css' => '67f4680d',
|
'rsrc/css/application/conpherence/menu.css' => '67f4680d',
|
||||||
'rsrc/css/application/conpherence/message-pane.css' => '50b1345e',
|
'rsrc/css/application/conpherence/message-pane.css' => '50b1345e',
|
||||||
|
@ -550,7 +550,7 @@ return array(
|
||||||
'conduit-api-css' => 'ce2cfc41',
|
'conduit-api-css' => 'ce2cfc41',
|
||||||
'config-options-css' => '16c920ae',
|
'config-options-css' => '16c920ae',
|
||||||
'conpherence-color-css' => 'b17746b0',
|
'conpherence-color-css' => 'b17746b0',
|
||||||
'conpherence-durable-column-view' => '2d57072b',
|
'conpherence-durable-column-view' => 'f68f35e3',
|
||||||
'conpherence-header-pane-css' => 'c9a3db8e',
|
'conpherence-header-pane-css' => 'c9a3db8e',
|
||||||
'conpherence-menu-css' => '67f4680d',
|
'conpherence-menu-css' => '67f4680d',
|
||||||
'conpherence-message-pane-css' => '50b1345e',
|
'conpherence-message-pane-css' => '50b1345e',
|
||||||
|
|
|
@ -311,6 +311,6 @@ img {
|
||||||
}
|
}
|
||||||
|
|
||||||
.minimize-column .conpherence-durable-column .phabricator-application-menu
|
.minimize-column .conpherence-durable-column .phabricator-application-menu
|
||||||
.fa-gear {
|
.conpherence-settings-icon {
|
||||||
display: none;
|
display: none;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue