Context Navigation

← Previous Ticket
Next Ticket →

#17390 closed enhancement (fixed)

Tiddly tiny little Settings screen tweaks

Reported by:	johnbillion	Owned by:	nacin
Priority:	normal	Milestone:	3.4
Component:	UI	Version:
Severity:	minor	Keywords:	has-patch commit
Cc:		Focuses:

Description

It bugs me when the description texts wrap off the edge on the Settings screens. Be nice if they had a linebreak after the input so they were a little more inline.

Attachments (1)

17390.patch (5.8 KB ) - added by johnbillion 15 years ago.

Download all attachments as: .zip

Change History (9)

@johnbillion
15 years ago

Attachment 17390.patch added

#1 @johnbillion
15 years ago

Keywords has-patch added

Patch

#2 @SergeyBiryukov
15 years ago

Keywords ui-feedback added

#3 @jane
15 years ago

Component Administration → UI
Keywords ui-feedback removed
Milestone Awaiting Review → 3.4

I haven't looked at the patch applied, but the description sounds reasonable. Fine by me (reserving right to change mind if once live it doesn't look good. :) )

#4 @nacin
14 years ago

Keywords commit added

Committing so Jane can see.

#5 @nacin
14 years ago

Would it make sense to just make span.description display:block and/or convert them to div's?

#6 @helenyhou
14 years ago

Wouldn't a p make sense for these?

#7 @nacin
14 years ago

Owner set to nacin
Resolution → fixed
Status new → closed

In [20605]:

Add line breaks before most form field descriptions (the italicized gray text on General Settings and other screens). props johnbillion, helenyhou, fixes #17390.

#8 @nacin
14 years ago

I used <br/><span class="description"> when it followed a textarea, as it caused extra margins that I couldn't track down.

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#17390 closed enhancement (fixed)

Tiddly tiny little Settings screen tweaks

Description

Attachments (1)

Change History (9)

@johnbillion 15 years ago

#1 @johnbillion 15 years ago

#2 @SergeyBiryukov 15 years ago

#3 @jane 15 years ago

#4 @nacin 14 years ago

#5 @nacin 14 years ago

#6 @helenyhou 14 years ago

#7 @nacin 14 years ago

#8 @nacin 14 years ago