Attachments (3)
Change History (15)
by , 14 years ago
Attachment: | Screen shot 2011-07-05 at 21.36.38.png added |
---|
comment:1 by , 14 years ago
Summary: | line height in relation editor it to big → line height in relation editor is to big |
---|
comment:2 by , 14 years ago
Summary: | line height in relation editor is to big → line height in relation editor is too big |
---|
comment:3 by , 14 years ago
follow-up: 5 comment:4 by , 14 years ago
that might be true, that sometimes the line height was too narrow in earlier versions, but that is no a reason to overshoot it by effectively doubling the line height. that is just a waste of screen estate. why don't you do it like in the normal tags editor in the right side pane? the cell height seems to be fine there to display all letters.
comment:5 by , 14 years ago
Replying to flaimo:
why don't you do it like in the normal tags editor in the right side pane? the cell height seems to be fine there to display all letters.
The table of the tags editor isn't editable. The rows dosn't need to reserve space for a cell editor.
I will limit the border size of the cell editor.
by , 14 years ago
Attachment: | 6547.patch added |
---|
initialize the cell editors with a small (1 pixel) empty border
comment:6 by , 14 years ago
Summary: | line height in relation editor is too big → [patch] line height in relation editor is too big |
---|
comment:9 by , 14 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
comment:10 by , 14 years ago
Sorry, I looked, but I didn't see the cell editor from the column model. Now the patch (6547-2.patch) reuse the the existing cell editor to change the border of the text field.
comment:12 by , 11 years ago
Description: | modified (diff) |
---|---|
Keywords: | relation font height added |
This is the result of the fix for #4948. Before this fix the row was to small for the cell editor.