WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 20 months ago

#17605 closed defect (bug) (fixed)

Inaccurate string in custom-header.php

Reported by: rasheed Owned by: westi
Milestone: 3.5 Priority: normal
Severity: normal Version: 3.1
Component: Customize Keywords: dev-feedback 3.3-early
Focuses: Cc:

Description

Hello,

Please check this string:
http://translate.wordpress.org/projects/wp/dev/ar/default?filters[status]=either&filters[original_id]=8092&filters[translation_id]=729698

I think it should be:

This will restore the original header text color. You will not be able to restore any customizations.

instead of:

This will restore the original header text. You will not be able to restore any customizations.

Attachments (2)

17605.diff (771 bytes) - added by scribu 3 years ago.
17605.2.patch (650 bytes) - added by ocean90 3 years ago.

Download all attachments as: .zip

Change History (19)

comment:1 ocean903 years ago

  • Keywords close added

It's not only the color. Also if the text should be displayed or not.

comment:2 scribu3 years ago

  • Keywords dev-feedback added; close removed

Could someone point me to a tutorial that displays the header text UI?

I wasn't able to reach step 2 of the process. Maybe it's dead code?

Version 0, edited 3 years ago by scribu (next)

comment:3 ocean903 years ago

  • Keywords dev-feedback removed

Why step 2?

if ( defined('HEADER_TEXTCOLOR') && get_theme_mod('header_textcolor') )

=> http://awesomescreenshot.com/05bdv9zdc

comment:4 scribu3 years ago

  • Component changed from General to Administration
  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to Future Release
  • Type changed from enhancement to defect (bug)

Ah, I was using Twentyten.

I think rasheed is right. The button is about resetting the text color.

For displaying the text, there's the "Display Text" radio buttons above.

scribu3 years ago

comment:5 scribu3 years ago

  • Keywords has-patch added; needs-patch removed
  • Milestone changed from Future Release to 3.2
  • Summary changed from inaccurate string to Inaccurate string in custom-header.php

Corrected both the label and button strings.

Sidenote: That entire screen has way too much text on it.

comment:6 ocean903 years ago

For displaying the text, there's the "Display Text" radio buttons above.

Yesno. If HEADER_TEXTCOLOR is blank then the header text is disabled. Also without JS you can't use the radio buttons.

comment:7 scribu3 years ago

The point is that clicking the "Restore Original Header Text Color" button (that's a mouth-full) won't and shouldn't affect wether the text is displayed or not.

comment:8 rasheed3 years ago

The problem there is no meaning matching between the title (Reset Text Color) and it's description (This will restore the original header text).

ocean903 years ago

comment:9 ocean903 years ago

  • Keywords dev-feedback added

The point is, that the button text is Restore Original Header Text and because of the function behind the button the inaccurate string is Reset Text Color, it should be simply Reset Text.

comment:10 scribu3 years ago

To clarify: "what does this button do?" (seriously)

comment:11 ocean903 years ago

I agree, it's a little bit unclear. Primary it removes the theme mod header_textcolor. So yes it's the color. BUT when HEADER_TEXTCOLOR is empty or has the value blank the "header text color will be transparent", means the header text won't be shown. Also the radio button will switch to No.

comment:12 westi3 years ago

  • Keywords needs-patch 3.3-early added; has-patch removed
  • Milestone changed from 3.2 to Future Release
  • Owner set to westi
  • Status changed from new to assigned
  • Version set to 3.1

The wording of a number of the bits of the custom header page is akward and I want to revisit them all in one go.

Moving from 3.2 and will re-consider in 3.3

comment:13 jkudish3 years ago

  • Keywords has-patch added; needs-patch removed

updating keywords

comment:14 SergeyBiryukov3 years ago

  • Keywords needs-patch added; has-patch removed

As per westi's comment above, this needs additional work.

comment:15 chipbennett3 years ago

  • Cc chip@… added

Related: #18887

comment:17 SergeyBiryukov20 months ago

  • Component changed from Administration to Appearance
  • Keywords needs-patch removed
  • Milestone changed from Future Release to 3.5
  • Resolution set to fixed
  • Status changed from assigned to closed

[22695] removed the button.

Note: See TracTickets for help on using tickets.