- Issue created by @utkarsh_33
- Status changed to Needs review
over 1 year ago 4:12am 11 August 2023 - last update
over 1 year ago 29,958 pass - ๐ฎ๐ณIndia gauravvvv Delhi, India
I have attached a patch for same, also added after patch screenshot for same. please review
After Patch
- Status changed to Needs work
about 1 year ago 6:37am 16 August 2023 - ๐ซ๐ฎFinland lauriii Finland
-
+++ b/core/themes/claro/css/theme/views_ui.admin.theme.pcss.css @@ -640,6 +640,11 @@ td.group-title { .views-list-section { ... + & .ajax-progress--throbber {
This problem isn't specific to the views list, this bug could happen on any table that uses operations with ajax. The reason this is more prone to happen on the views list table is that it has a lot of columns and long description texts, making the operations column less likely to have additional space for the throbber.
-
+++ b/core/themes/claro/css/theme/views_ui.admin.theme.pcss.css @@ -640,6 +640,11 @@ td.group-title { + position: absolute; + margin-block-start: 5px;
It might be acceptable to use position: absolute but there might be better approaches. The key drawback with this approach is that when there isn't a lot of space (like on a narrow screen), the throbber might be pushed outside of the table.
-
- Status changed to Needs review
about 1 year ago 11:06am 30 August 2023 - last update
about 1 year ago Custom Commands Failed - ๐ฎ๐ณIndia swatidhurandhar
This issue is due to a lack of space in the table columns; if the operations column width is not enough, then the progress throbber moves to the next line. I have checked other tables, like on the content page, where throbber comes next to operation links since there is enough space.
All the columns in the views page table have a width defined, which is added in the views UI module. So I changed the width of two columns in theme to provide enough space for the throbber icon, which then moved to the right side of the operations button.
- Status changed to Needs work
about 1 year ago 5:00pm 30 August 2023 - ๐ฎ๐ณIndia gauravvvv Delhi, India
In patch #4, I think we should not fix the width of table columns.
- ๐ฎ๐ณIndia swatidhurandhar
The ajax throbber has global styling which is working for all the tables. The reason for throbber coming to next line was due to the space issue that's why reduced the width of column in views table.
- last update
12 months ago 30,604 pass - Assigned to rupeshghar
- Issue was unassigned.
- ๐ฎ๐ณIndia akashdab
As highlighted by many before, there is a lack of space after the Edit dropdown to accommodate the throbber inline.
As it stays right now, the entire Edit dropdown field is 128px in width, of which 96px is the width of the dropdown element, 32 px is the width shared onto either sides of the dropdown.
To successfully include the throbber inline there needs to be at least 42px space after the dropdown, as that is the width the throbber takes.To tackle this there are 3 approaches we can take,
- Involves redistributing the width taken by the table header.
- Involves redistributing the width taken by the table header, and
- 2(a). reducing padding of the Edit dropdown element.
2(b). reducing padding of the Edit dropdown element and reducing margin taken by ajax throbber.
1. Redistributing the width taken by the table header.
- Reduce the width of View name table header from 20% to 15%.
- In turn allocate the free space to Operations table header, increasing it from 10% to 15%.
- Doing this increases the width of Edit dropdown from 96px to approximately 145px, consequently making enough space for the throbber.
- Method 1 Plain View
-
Method 1 Calculated
2(a). Redistributing the width from the table header and reducing padding of edit dropdown.
- This one takes ideas from patch-4, reducing the width of View name table header from 20% to 17%.
- Allocating the free space to Operations TH, increasing its width from 10% to 13%.
- Reducing the padding from TD of Operations TH, from 16px to 7px (or .45 rem ~ 7px)
-
Method 2(a) Plain View
-
Method 2(a) Calculated
2(b). Redistributing the width from the table header, reducing padding of edit dropdown and reducing margin taken by ajax throbber.
- Reducing the width of View name table header from 20% to 17%.
- Allocating the free space to Operations TH, increasing its width from 10% to 13%.
- Reducing the padding from TD of Operations TH, from 16px to 10px (or .625rem)
- Reducing the margin from ajax throbber from 12px to 9px (0.5625rem).
Method 2(b) Plain View
Method 2(b) Calculated
- ๐ฎ๐ณIndia hetal.solanki
Hetal.Solanki โ made their first commit to this issueโs fork.