![Announcement icon Announcement icon](https://www.drupal.org/files/styles/grid-2-2x-square/public/announcements/drupal-evergreen-logo-280X280px%20%281%29_0.jpg?itok=PHpn6rCb)
Text between <code> tags is displayed in a very small font size. The attached patch increases this size to 150%, making the code look identical in size to regular text. Tested in Safari 3.2.1 and Firefox 3.0.8 (Mac).
Comment | File | Size | Author |
---|---|---|---|
#15 | 421076_code_font-size_seven.png | 15.13 KB | stBorchert |
#10 | 421076_code_font_size-10.patch | 599 bytes | stBorchert |
#9 | font_size_code_before.png | 2.05 KB | stBorchert |
#9 | font_size_code_after.png | 1.91 KB | stBorchert |
#4 | code_font_size_02.patch | 809 bytes | Xano |
![Acquia logo](/files/ads/acquia-issue-sponsorship.png)
Comments
Comment #1
sunsubscribing
Comment #2
XanoShort notice: this needs a per-theme approach.
Comment #3
XanoComment #4
XanoFollow-up that also increases font-size for PRE and KBD elements.
Comment #5
Dries CreditAttribution: Dries commentedDo you have an explanation for why we need to set it to 150% rather than have it magically work at the default size? I guess I could fire up Firebug and find out ...
Comment #6
XanoThe default font size of these elements is smaller than the font size of the surrounding text, making the code hard to read. This patch makes all font sizes equal.
Comment #7
sunHm. Why 150% ?
Usually, I have this default snippet in all of my projects' themes:
Whereas a popular browser CSS reset snippet does:
...which is equally only slightly larger than 100%.
Comment #8
catchThis either needs a comment or a different size.
Comment #9
stBorchertI'm ok with
115%
.The first screenshot shows the actual size and the second with size set to
115%
.Comment #10
stBorchertRe-roll with
font-size: 115%;
.Comment #12
deekayen CreditAttribution: deekayen commentedinfant bot doesn't know how to test patches properly yet
Comment #13
Bojhan CreditAttribution: Bojhan commentedMarking this RTBC, tried it out, and it looks good.
Comment #14
sunWow, this issue still around? - definitely.
Though - shouldn't this be applied to all core themes? (not limited to Garland)
Comment #15
stBorchertSeven doesn't need this modification because its looking good (see screenshot).
Comment #16
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD. Thanks!