#6547 closed defect (fixed)
[patch] line height in relation editor is too big
Reported by: | flaimo | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | Core | Version: | tested |
Keywords: | relation font height | Cc: |
Description (last modified by )
Attachments (3)
Change History (15)
Changed 12 years ago by
Attachment: | Screen shot 2011-07-05 at 21.36.38.png added |
---|
comment:1 Changed 12 years ago by
Summary: | line height in relation editor it to big → line height in relation editor is to big |
---|
comment:2 Changed 12 years ago by
Summary: | line height in relation editor is to big → line height in relation editor is too big |
---|
comment:3 Changed 12 years ago by
comment:4 follow-up: 5 Changed 12 years ago by
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 Changed 12 years ago by
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.
Changed 12 years ago by
Attachment: | 6547.patch added |
---|
initialize the cell editors with a small (1 pixel) empty border
comment:6 Changed 12 years ago by
Summary: | line height in relation editor is too big → [patch] line height in relation editor is too big |
---|
comment:9 Changed 12 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
comment:10 Changed 12 years ago by
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 Changed 8 years ago by
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.