Hi @mmacfarlane
I created a POC to test whether we can let users configure whether a table should have sticky headers or not. Here's a demo:
Add sticky table headers for markdown tables in view mode
Wrap markdown tables with a bounded-height scroll container (max-height: 70vh) and apply position: sticky to thead cells. This keeps column headers visible when scrolling long tables.
Changelog: added
| Before | After |
|---|---|
Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Related to #585265
Vanessa Otto (9ba0903c) at 18 Mar 16:38
POC: Editor sticky header configuration
... and 5 more commits
but displayPagination is not a method anymore
Morning @lindsey-shelton, would you have time for a small MR review?
Vanessa Otto (8f178373) at 18 Mar 14:58
Align footer padding bottom of sessions with duo chat drawer
Align footer padding bottom of sessions with duo chat drawer
| Before | After |
|---|---|
![]() |
![]() |
ee/app/assets/javascripts/ai/duo_agents_platform/components/common/agent_flow_list.vue to see the pagination)Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Related to #593997
Vanessa Otto (944697e0) at 18 Mar 14:55
Align footer padding bottom of sessions with duo chat drawer
Vanessa Otto (ac6d7747) at 18 Mar 14:47
merci! =)
@danmh Caution
d9b3b213: Use full URLs instead of short references (
gitlab-org/gitlab#123or!123), as short references are displayed as plain text outside of GitLab. For more information, take a look at our Commit message guidelines.
Update progress bar to use the full border radius design token (gl-rounded-full / 9999px) instead of the default radius, to match the fully-rounded appearance shown #2565.
This follows up on !5753 which introduced gl-rounded-default, but the approved design in #2565 calls for a fully-rounded border radius.
Visual change: the progress bar track will have fully-rounded ends (border-radius: 9999px) instead of the default 0.25rem.
Relates to #2565 (border-radius portion)
now that makes more sense =D
Hi @nickbrandt, I have a question regarding the table padding in editors. Apologies for disturbing you already after Alex is PTO for just a day =D This is absolutely non-urgent.
In the screenshots in the work item description, you can see that tables in view or preview mode have a more condensed vertical padding of 10px (padding: 10px 16px;), while in the RTE it's 16px all around. The RTE style matches the Pajamas style. However, this might be intentional because tables can get quite large and we'd like to see more data in less space. In that case, I wonder if we should also use the reduced padding in the RTE editor? There is a chevron, though, that would get less space as well:
| RTE current padding | RTE reduced padding |
|---|---|
![]() |
![]() |
Vanessa Otto (ab7910ee) at 18 Mar 10:41
Gate content editor table sticky headers behind feature flag
... and 8 more commits
Vanessa Otto (88d5dbec) at 18 Mar 10:36
Does not set max height in print mode
... and 6 more commits
Vanessa Otto (03b13230) at 18 Mar 10:03
Wrap content editor table in a scrollable div with sticky header su...
... and 297 more commits