lang icon indicating copy to clipboard operation
lang copied to clipboard

[uk]: Fix apostrophes

Open lopatin96 opened this issue 1 year ago • 1 comments

Repalce apostrophe "'" with "’", because the first one in some places are displayed as "'"

lopatin96 avatar Apr 27 '24 22:04 lopatin96

Please accept the changes, because now there is a mistake during registration: Screenshot 2024-04-28 at 00 39 19

lopatin96 avatar Apr 27 '24 22:04 lopatin96

Hi, @lopatin96, how do you display translations?

In general, I'm not against it.

Oleksandr-Moik avatar Apr 28 '24 12:04 Oleksandr-Moik

@Oleksandr-Moik here is register.blade.php out of the box, so the way is __('Name') Screenshot 2024-04-28 at 14 46 21

lopatin96 avatar Apr 28 '24 12:04 lopatin96

Ok. I will debug this.

Oleksandr-Moik avatar Apr 28 '24 13:04 Oleksandr-Moik

There is some issue with blade components. If we use it without changes, apostrophe will be converted to a html code. image

But with dd, we see the value already encoded image

Yes, character replacement fixes the problem, but it is not a good solution. image

Because when we use another format, it passes without problems image


In my opinion, this is a case better fixed on the project side, not the package side.


@andrey-helldar, I need your help — We change it only in general file? What to do with the rest of the translations?

If ok, then @lopatin96 will replace the apostrophe also in php.json and php-inline.json to keep the same format.

Oleksandr-Moik avatar Apr 28 '24 13:04 Oleksandr-Moik

@lopatin96, @Oleksandr-Moik

In other locales it is better to leave it as it is otherwise we may encounter using the wrong character.

Instead, I can recommend setting "smart punctuation" in the application: https://laravel-lang.com/configuration.html#smart_punctuation

andrey-helldar avatar Apr 28 '24 13:04 andrey-helldar

Я обдумав всі варіант і вагаюся. Ці зміни можна застосувати, але залишу хід своїх думок:

Я, наприклад, до сьогодні не знав, де і як можна поставити з клавіатури ПК/ноута апостроф) Також якщо доведеться додавати ще переклад з апострофом (в проект чи нам у пакет), потрібно згадати, що потрібно використати інший символ. Не хочеться, щоб в межах проекту був мікс таких форматів. Насправді легше використати інший формат передачі/ренедеру перекладу, ніж замінювати.

Та й просто, вже якось занадто звиклося використовувати та бачити ' як апостроф)

Oleksandr-Moik avatar Apr 28 '24 14:04 Oleksandr-Moik

Я схиляюся до того, щоб залишити поточну версію коду без внесення змін, тому що якщо додати у ваш застосунок, наприклад, французьку мову, то зіткнетеся з тією ж проблемою.

Вирішити це найпростіше активацією функції "розумної пунктуації" у вашому застосунку.

А ще під час додавання нових ключів 100% матиме місце людський фактор, тому в нових ключах ми з великою часткою ймовірності можемо забути змінювати апострофи.

andrey-helldar avatar Apr 29 '24 03:04 andrey-helldar