Opened 4 years ago

Closed 4 years ago

#13686 closed Bug (invalid)

Find and replace text percentage is not correct for Arabic locale

Reported by: ACGC Owned by:
Priority: Normal Milestone:
Component: UI : Language Version: 4.5.3
Keywords: Cc:

Description

Steps to reproduce

  1. Write a word and copy/paste it multiple times.
  2. Open the search dialog
  3. Write the word in the find field and another word in the replace field.
  4. Click on 'Replace all'

Expected result

100% of the words were replaced (image1)

Actual result

The replace dialog shows that 1% of the words have been replaced (image2)

Other details (browser, OS, CKEditor version, installed plugins)

Attachments (2)

image1.png (54.1 KB) - added by ACGC 4 years ago.
expected behaviour 100% of the words were replaced.
image2.png (77.5 KB) - added by ACGC 4 years ago.
actual behaviour where replace dialog shows that 1% of the words have been replaced

Download all attachments as: .zip

Change History (4)

Changed 4 years ago by ACGC

Attachment: image1.png added

expected behaviour 100% of the words were replaced.

Changed 4 years ago by ACGC

Attachment: image2.png added

actual behaviour where replace dialog shows that 1% of the words have been replaced

comment:1 Changed 4 years ago by ACGC

I provided solution and created pull request: https://github.com/ckeditor/ckeditor-dev/pull/210

comment:2 Changed 4 years ago by Anna Tomanek

Component: GeneralUI : Language
Resolution: invalid
Status: newclosed

Thank you for taking your time to investigate this.

However, "%1" is actually a placeholder to display the number of occurances replaced. It should be replaced by CKEditor with an actual number, just like it works for other language versions, and not be replaced with 100%. For some reason it does not get replaced in Arabic and I created ticket #13688 for this issue.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy