Skip to content

Commit

Permalink
[WEB-566] feat: Added text to empty color picker boxes and other fixes (
Browse files Browse the repository at this point in the history
#3867)

* fix: z index issues with modals and on hover color in table item picker menu

* feat: added text indicators inside the table colors to give a gist of how text would look
  • Loading branch information
Palanikannan1437 authored Mar 6, 2024
1 parent 69fa170 commit e1db39f
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 6 deletions.
6 changes: 3 additions & 3 deletions packages/editor/core/src/styles/table.css
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@
top: 0;
bottom: -2px;
width: 4px;
z-index: 99;
z-index: 5;
background-color: #d9e4ff;
pointer-events: none;
}
Expand All @@ -111,7 +111,7 @@
.tableWrapper .tableControls .rowsControl {
transition: opacity ease-in 100ms;
position: absolute;
z-index: 99;
z-index: 5;
display: flex;
justify-content: center;
align-items: center;
Expand Down Expand Up @@ -198,7 +198,7 @@

.tableWrapper .tableControls .tableToolbox .toolboxItem:hover,
.tableWrapper .tableControls .tableColorPickerToolbox .toolboxItem:hover {
background-color: rgba(var(--color-background-100), 0.5);
background-color: rgba(var(--color-background-80), 0.6);
}

.tableWrapper .tableControls .tableToolbox .toolboxItem .iconContainer,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -213,10 +213,11 @@ function createToolbox({
{ className: "colorPicker grid" },
Object.entries(colors).map(([colorName, colorValue]) =>
h("div", {
className: "colorPickerItem",
className: "colorPickerItem flex items-center justify-center",
style: `background-color: ${colorValue.backgroundColor};
color: ${colorValue.textColor || "inherit"};`,
innerHTML: colorValue?.icon || "",
color: ${colorValue.textColor || "inherit"};`,
innerHTML:
colorValue.icon ?? `<span class="text-md" style:"color: ${colorValue.backgroundColor}>A</span>`,
onClick: () => onSelectColor(colorValue),
})
)
Expand Down

0 comments on commit e1db39f

Please sign in to comment.