fix bug in icon removal in deactivation. Fix bottom border display regression

pull/1671/head
Joseph Izang 3 years ago
parent 9dfb20757f
commit 87f4e06fba
  1. 4
      apps/remix-ide/src/app/components/vertical-icons.js
  2. 6
      libs/remix-ui/vertical-icons-panel/src/lib/remix-ui-vertical-icons-panel.css
  3. 4
      libs/remix-ui/vertical-icons-panel/src/lib/remix-ui-vertical-icons-panel.tsx

@ -123,7 +123,9 @@ export class VerticalIcons extends Plugin {
*/
removeIcon ({ name }) {
if (this.targetProfileForChange[name]) delete this.targetProfileForChange[name]
this.renderComponent()
setTimeout(() => {
this.renderComponent()
}, 150)
}
/**

@ -84,7 +84,7 @@
}
.remixui_scrollable-container {
max-height: 570px;
border-bottom: 3px solid #3f4455;
/* border-bottom: 3px solid #3f4455; */
}
.remixui_scrollbar::-webkit-scrollbar { /* Chrome, Safari and other Webkit browsers*/
display: none;
@ -99,6 +99,10 @@
z-index: 1000;
}
.remixui_settings {
min-height: 30px;
}
#menuitems {
list-style: none;
margin: 0px;

@ -316,8 +316,8 @@ export function RemixUiVerticalIconsPanel ({
</div>
<div
id="remixuiScrollable"
className={scrollableRef.current && scrollableRef.current.clientHeight > 0
? 'border-bottom remixui_default-icons-container remixui_scrollable-container remixui_scrollbar remixui_hide-scroll'
className={scrollableRef.current && scrollableRef.current.scrollHeight > scrollableRef.current.clientHeight
? 'remixui_default-icons-container remixui_scrollable-container remixui_scrollbar remixui_hide-scroll'
: 'remixui_scrollable-container remixui_scrollbar remixui_hide-scroll'}
ref={scrollableRef}
>

Loading…
Cancel
Save