Context Navigation

← Previous Ticket
Next Ticket →

#7387 closed defect (bug) (fixed)

Press This Text textarea should have height defined

Reported by:	zamoose	Owned by:	noel
Priority:	low	Milestone:	2.6.1
Component:	Administration	Version:	2.6
Severity:	normal	Keywords:	press-this, textarea, css, formatting
Cc:		Focuses:

Description

The current implementation of the posting textarea in the Press This interface has a width specified, but no height. This results in extreme ugliness and nigh-unusability, particularly in Gecko-powered browsers. To wit, this screencap shows how, when a block of text is highlighted and Press This is clicked, the text itself is pretty much useless.

The textarea ought to have a rows="" or similar to define a minimum height of at least 5 lines, IMNSHO.

Attachments (1)

7387.diff (903 bytes ) - added by zamoose 18 years ago.: Simple diff adding rows=""

Download all attachments as: .zip

Change History (7)

#1 @ryan
18 years ago

Owner changed from anonymous to noel

Note: This is seen when the RTE is disabled. With RTE it looks good.

#2 @zamoose
18 years ago

Ahhh, yes, Ryan is correct. I do have the RTE disabled.

@zamoose
18 years ago

Attachment 7387.diff added

Simple diff adding rows=""

#3 @zamoose
18 years ago

Here's a simple diff that adds rows="10" to the textarea affected. Looks great in my non-RTE-enabled setup, though 15 could be even better.

#4 @noel
18 years ago

Fix is attached to ticket:7310

#5 @ryan
18 years ago

Resolution → fixed
Status new → closed

(In [8551]) Press This: Fix image insert in IE7, set textarea height. Props noael and zamoose. fixes #7387 #7310 for trunk

#6 @ryan
18 years ago

(In [8552]) Press This: Fix image insert in IE7, set textarea height. Props noael and zamoose. fixes #7387 #7310 for 2.6

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#7387 closed defect (bug) (fixed)

Press This Text textarea should have height defined

Description

Attachments (1)

Change History (7)

#1 @ryan 18 years ago

#2 @zamoose 18 years ago

@zamoose 18 years ago

#3 @zamoose 18 years ago

#4 @noel 18 years ago

#5 @ryan 18 years ago

#6 @ryan 18 years ago