Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

k-table: index column too narrow for high indexes #6451

Open
distantnative opened this issue May 18, 2024 · 2 comments
Open

k-table: index column too narrow for high indexes #6451

distantnative opened this issue May 18, 2024 · 2 comments
Assignees
@afbora
Labels
type: bug 🐛 Is a bug; fixes a bug

Comments

@distantnative
Copy link
Member

4 digit indexes cannot be displayed:

Screenshot 2024-05-18 at 19 11 07

First index is actually 1201
@distantnative distantnative added the type: bug 🐛 Is a bug; fixes a bug label May 18, 2024
@afbora
Copy link
Member

afbora commented Jun 1, 2024

@distantnative I think if we remove the padding-inline for first index, the issue will be solved.

kirby/panel/src/components/Layout/Table.vue

Line 386 in c1e6ff7

padding-inline: var(--table-cell-padding);

@afbora afbora self-assigned this Jun 3, 2024
afbora added a commit that referenced this issue Jun 3, 2024
@afbora
Fix table index column for high indexes #6451
869bee0
@afbora afbora mentioned this issue Jun 3, 2024
Closed
5 tasks
@afbora afbora added this to the 4.3.0 milestone Jun 3, 2024
@distantnative distantnative modified the milestones: 4.3.0, 4.4.0 Jun 4, 2024
@distantnative distantnative removed this from the 4.4.0 milestone Jul 22, 2024
@distantnative
Copy link
Member Author

@afbora I think the culprit is rather

kirby/panel/src/components/Layout/Table.vue

Line 480 in c1e6ff7

width: var(--table-row-height);

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@afbora afbora

Labels
type: bug 🐛 Is a bug; fixes a bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@afbora @distantnative