Hi,
Look at the picture below, there is a display problem with link module in my node type.
Thanks
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | link_display.patch | 529 bytes | quicksketch |
| #2 | link_display_problem.png | 4.82 KB | mrfelton |
| #2 | link_display.patch | 448 bytes | mrfelton |
| linkissue.png | 34.63 KB | Julien PHAM |
Comments
Comment #1
quicksketchLink module does it's best to provide suitable default CSS (it should look okay in Drupal's core themes), but it looks like in your theme you'll need to add a few lines to reduce the width of your textfields.
Try adding something like this to your theme's style.css.
Comment #2
mrfelton commentedI found that even with the default garland theme, whilst the link field displayed ok on a node creation form, it didn't display properly in the Default Value fieldset in the settings form for a link field. The current link.css file does alter the size of the containing div, however, by default input fields are pretty long, resulting in the two fields actually overlapping in the Default Value fieldset on a link field configuration page (see attached image).
I think we need to include a little more css as this is a problem even in the default theme.
The attached patch seems to do the trick.
Comment #3
mrfelton commentedComment #4
quicksketchThis looks great. Thanks for nailing down the situation mrfelton, I wasn't sure what the use-case was. While browser checking the patch I found that our CSS didn't ever work at all in IE 6/7 (or at least it displayed on separate lines). Tweaking the column width to slightly more narrow fixes this in IE also.
Comment #5
bcobin commentedPatch didn't apply properly in 6.x, but it's easy enough to edit the CSS manually. Works like a charm! (have yet to test in IE6/7 but I think it should be OK.
Thanks much!
Comment #6
hanno commentedThis simple patch fix a problem for me as well but is not included in the css of the module yet. is it possible to include this small patch in the next update of link.module?
div.link-field-column .form-text {
width: 95%;
}
Comment #7
jcfiala commentedAlright, I've included this item in cvs and it will be in the next version of the 6.x branch. (2.7, as long as the number system does not fail us.)