Archives/gristlabs_grist-core
1
0
Fork 0
mirror of https://github.com/gristlabs/grist-core.git synced 2024-10-27 20:44:07 +00:00
Code Issues Projects Releases Wiki Activity

(core) Revert Safari overflow fix

Browse Source 
Summary: The fix introduced a regression.

Test Plan: Existing tests.

Reviewers: jarek

Reviewed By: jarek

Differential Revision: https://phab.getgrist.com/D4254
This commit is contained in:
George Gevoian 2024-05-15 11:49:09 -07:00
parent 3267012226
commit ed9514bae0
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/client/ui2018/menus.ts
View File

@ -166,7 +166,8 @@ const cssSubMenuElem = styled('div', `
min-width: 200px; min-width: 200px;
`); `);
// Note: "overflow: visible" happens to matter for Safari. // TODO: menus are sometimes cut off in Safari.
// Setting "overflow: visible fixes this, but breaks overflow elsewhere.
export const cssMenuElem = styled('div', ` export const cssMenuElem = styled('div', `
font-family: ${vars.fontFamily}; font-family: ${vars.fontFamily};
font-size: ${vars.mediumFontSize}; font-size: ${vars.mediumFontSize};
@ -179,7 +180,6 @@ export const cssMenuElem = styled('div', `
--weaseljs-selected-background-color: ${theme.menuItemSelectedBg}; --weaseljs-selected-background-color: ${theme.menuItemSelectedBg};
--weaseljs-menu-item-padding: 8px 24px; --weaseljs-menu-item-padding: 8px 24px;
background-color: ${theme.menuBg}; background-color: ${theme.menuBg};
overflow: visible;
@media print { @media print {
& { & {