Browse Source

fix(nc-gui): PR review changes #2417

pull/7741/head
Ramesh Mane 9 months ago
parent
commit
776454cc4e
  1. 11
      packages/nc-gui/components/cell/RichText/SelectedBubbleMenu.vue
  2. 7
      packages/nc-gui/components/project/AllTables.vue
  3. 10
      packages/nc-gui/components/smartsheet/form/LimitOptions.vue

11
packages/nc-gui/components/cell/RichText/SelectedBubbleMenu.vue

@ -79,15 +79,12 @@ const onToggleLink = () => {
<template>
<div
class="bubble-menu flex-row gap-x-1 py-1 rounded-lg"
:class="[
`${isFormField ? 'inline-flex !bg-transparent' : 'flex bg-gray-100 px-1'}`,
{
:class="{
'inline-flex !bg-transparent': isFormField,
'flex bg-gray-100 px-1': !isFormField,
'embed-mode': embedMode,
},
{
'full-mode': !embedMode,
},
]"
}"
>
<NcTooltip :placement="isFormField ? 'bottom' : undefined" :disabled="editor.isActive('codeBlock')">
<template #title>

7
packages/nc-gui/components/project/AllTables.vue

@ -80,7 +80,12 @@ const onCreateBaseClick = () => {
<template>
<div class="nc-all-tables-view">
<div class="flex flex-row gap-x-6 pb-3 pt-6" :class="base?.isLoading ? 'pointer-events-none' : ''">
<div
class="flex flex-row gap-x-6 pb-3 pt-6"
:class="{
'pointer-events-none': base?.isLoading,
}"
>
<div
v-if="isUIAllowed('tableCreate')"
role="button"

10
packages/nc-gui/components/smartsheet/form/LimitOptions.vue

@ -127,14 +127,12 @@ const showOrHideAll = (showAll: boolean) => {
if (props.isRequired && !showAll) {
return
}
vModel.value = [
...vModel.value.map((o) => {
vModel.value = vModel.value.map((o) => {
return {
...o,
show: showAll,
}
}),
]
})
}
</script>
@ -170,7 +168,7 @@ const showOrHideAll = (showAll: boolean) => {
<NcButton
type="secondary"
size="xxsmall"
class="!border-none !px-2 !text-xs !text-gray-500 disabled:!text-gray-300"
class="!border-none !px-2 !text-xs !text-gray-500 !disabled:text-gray-300"
:disabled="isRequired || vModel.filter((o) => !o.show).length === vModel.length"
@click="showOrHideAll(false)"
>
@ -182,7 +180,7 @@ const showOrHideAll = (showAll: boolean) => {
<NcButton
type="secondary"
size="xxsmall"
class="!border-none !px-2 !text-xs !text-gray-500 disabled:!text-gray-300"
class="!border-none !px-2 !text-xs !text-gray-500 !disabled:text-gray-300"
:disabled="vModel.filter((o) => o.show).length === vModel.length"
@click="showOrHideAll(true)"
>

Loading…
Cancel
Save