Context Navigation

← Previous Ticket
Next Ticket →

#18881 closed defect (bug) (fixed)

"Scale images to match ..." in "/wp-admin/includes/media.php"

Reported by:	msi08	Owned by:	azaozz
Priority:	normal	Milestone:	3.3
Component:	Media	Version:	3.3
Severity:	normal	Keywords:	has-patch
Cc:		Focuses:

Description

That sentence needs a better style. It's nearly impossible to create a good translation (meaning: it should sound right) because the word order in my native language is different.

Attachments (5)

media.php.diff (768 bytes ) - added by msi08 15 years ago.: media.php.diff
18881.patch (1.2 KB ) - added by SergeyBiryukov 15 years ago.
18881.2.patch (1.1 KB ) - added by SergeyBiryukov 15 years ago.
18881.3.patch (1.1 KB ) - added by ocean90 15 years ago.: add translators comment
18881.4.patch (1.7 KB ) - added by ocean90 15 years ago.

Download all attachments as: .zip

Change History (13)

@msi08
15 years ago

Attachment media.php.diff added

media.php.diff

#1 follow-up: ↓ 3 @ocean90
15 years ago

Component General → Media
Keywords needs-patch added
Milestone Awaiting Review → 3.3
Type enhancement → defect (bug)
Version → 3.3

We need also a capability check here, don't show the link if the user can't manage_options.

@SergeyBiryukov
15 years ago

Attachment 18881.patch added

@SergeyBiryukov
15 years ago

Attachment 18881.2.patch added

#2 @SergeyBiryukov
15 years ago

Keywords has-patch added; needs-patch removed

We can also probably merge the strings into one.

Shouldn't it be "media settings" rather than "image options"?

#3 in reply to: ↑ 1 @azaozz
15 years ago

Replying to ocean90:

We need also a capability check here, don't show the link if the user can't manage_options.

Actually that link can be removed. Put it there as during testing the users were wondering where the 1024x1024 thing came from. Perhaps we can think of a better text/explanation that won't need the link.