Get it more user friendly, it should match the buttons name on editors itself.

eg.

Ckeditor:
Strike-through -> Strike through
Forecolor -> Text Color
Backcolor -> Background Color
etc....

Im willing to submit a patch but which GIT branch should I working on.

CommentFileSizeAuthor
#4 1230708-4.patch4.58 KBrpayanm
#2 buttons.patch4.35 KBdroplet
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

TwoD’s picture

TwoD
Primary language Swedish
Location Sweden
CreditAttribution: TwoD commented

We always patch master first and then backport to other branches when it's done. (We'll soon switch to using a 7.x-2.x branch instead but we haven't worked out exactly how to treat the by then old master branch.)

droplet’s picture

droplet CreditAttribution: droplet commented
Status: Active » Needs review
FileSize
buttons.patch4.35 KB

Patched 2 big brother editors.

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented
Issue summary: View changes
Status: Needs review » Needs work
Issue tags: +Needs reroll
Parent issue: » #2343445: Stable release of Wysiwyg module, v7.x-2.4 please
rpayanm’s picture

rpayanm
he/him
Primary language Spanish
CreditAttribution: rpayanm at SeeD EM commented
Status: Needs work » Needs review
Issue tags: -Needs reroll
FileSize
1230708-4.patch4.58 KB

  • TwoD committed 008e2be on 6.x-2.x authored by rpayanm 
    - #1230708 by droplet, rpayanm: Fixed mismatched button names.
  • TwoD committed 4210e74 on 7.x-2.x authored by rpayanm
    - #1230708 by droplet, rpayanm: Fixed mismatched button names.
TwoD’s picture

TwoD
Primary language Swedish
Location Sweden
CreditAttribution: TwoD as a volunteer commented
Title: some Buttons name in Wysiwyg do not match editor buttons name » Mismatched button names
Status: Needs review » Fixed

These were likely not the only ones, but it's a good start.
I am truly sorry it took so long to get around to this.

Thank you for patching, reviewing and re-rolling this!

TwoD’s picture

TwoD
Primary language Swedish
Location Sweden
CreditAttribution: TwoD as a volunteer commented

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.