Last modified: 2013-12-12 19:43:26 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T53314, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 51314 - VisualEditor: Create toolbar icons for bold/italic/underline/superscript/subscript/etc. for scripts/languages as appropriate (tracker)
VisualEditor: Create toolbar icons for bold/italic/underline/superscript/subs...
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
Editing Tools (Other open bugs)
unspecified
All All
: Normal minor
: ---
Assigned To: Rob Moen
: i18n
Depends on: 51330 51702 51703 51704 51705 51706 51707 51709 51710 51711 51712 51713
Blocks: tracking ve-nonenglish ve-tracking
  Show dependency treegraph
 
Reported: 2013-07-14 10:55 UTC by Mormegil
Modified: 2013-12-12 19:43 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Mormegil 2013-07-14 10:55:18 UTC
Since bug 38551 have been resolved, it is _possible_ to have per-language bold/italic button variants. But the only two languages _using_ this are currently English and German. Do we really need to ask language by language, when we've been through all this with the Vector edit toolbar (see e.g. bug 23625, bug 23911, bug 24188 etc.). We already have https://commons.wikimedia.org/wiki/WikiEditor_Toolbar_Icons#List_by_language -- cannot we use that?
Comment 1 James Forrester 2013-07-14 14:41:19 UTC
(In reply to comment #0)
> Since bug 38551 have been resolved, it is _possible_ to have per-language
> bold/italic button variants.

That's about Vector, which is entirely irrelevant here.

> But the only two languages _using_ this are currently English and German.

Yes (and the Latin-generic ones).

> Do we really need to ask language by language, when we've been through
> all this with the Vector edit toolbar (see e.g. bug 23625, bug 23911,
> bug 24188 etc.).

See above.

> We already have
> https://commons.wikimedia.org/wiki/WikiEditor_Toolbar_Icons#List_by_language
> --
> cannot we use that?

No. Those are low-quality PNG-only icons and not in keeping with the existing VisualEditor ones. VisualEditor's interface is in SVG. We look forward to the creation of such icons and would gladly include them. Turning this into a tracking bug so each language can be tracked against it.
Comment 2 Mormegil 2013-07-14 20:40:54 UTC
(In reply to comment #1)
> (In reply to comment #0)
> > Since bug 38551 have been resolved, it is _possible_ to have per-language
> > bold/italic button variants.
> 
> That's about Vector, which is entirely irrelevant here.

Really? Bug 38551 "VisualEditor: Bold and Italic UI buttons need to be internationalisable" fixed by https://git.wikimedia.org/commit/mediawiki%2Fextensions%2FVisualEditor/f49d80d27a757d0500b542959284a6c6bc99aebe? Anyway, that's not important.

> Yes (and the Latin-generic ones).

Which are wrong, that's the point.

> > We already have
> > https://commons.wikimedia.org/wiki/WikiEditor_Toolbar_Icons#List_by_language
> > --
> > cannot we use that?
> 
> No. Those are low-quality PNG-only icons and not in keeping with the existing
> VisualEditor ones. VisualEditor's interface is in SVG. We look forward to the
> creation of such icons and would gladly include them. Turning this into a
> tracking bug so each language can be tracked against it.

That was exactly my point is that we do not want to do that, because we have already went through this with Vector. But I guess I have not expressed myself clearly: I do not care about the exact image files linked there. The point is that _we already know_ that Czech, Hebrew, and Polish (in addition to English) want to use B/I, and we already have these images, the only problem is to modify the configuration. We already know that Danish, Norwegian, and Swedish (in addition to German) want to use F/K, and we already have these images, the only problem is to modify the configuration. We also already know what letters 13 other languages would want to use, even though we do not have those images ready (I guess), but the best person to create those missing images is the person who created the current A/A/B/I/F/K ones, I would say.

So: I believe I do not need to manually create 6 new bugs to request Czech, Hebrew, and Polish to have ve.ui.BoldButtonTool.static.icon switched to 'bold-b' and ve.ui.ItalicButtonTool.static.icon switched to 'italic-i', and Danish, Norwegian, and Swedish to have ve.ui.BoldButtonTool.static.icon switched to 'bold-f' and ve.ui.ItalicButtonTool.static.icon switched to 'italic-k'.
Comment 3 James Forrester 2013-07-14 22:30:14 UTC
(In reply to comment #2)
> That was exactly my point is that we do not want to do that, because we have
> already went through this with Vector. But I guess I have not expressed
> myself clearly: I do not care about the exact image files linked there. The
> point is that _we already know_ that Czech, Hebrew, and Polish (in addition
> to English) want to use B/I, and we already have these images, the only
> problem is to modify the configuration. We already know that Danish,
> Norwegian, and Swedish (in addition to German) want to use F/K, and we
> already have these images, the only problem is to modify the configuration.

Thank you! No, "we" do not know, as we are not experts - but you clearly do. :-) Have created this as bug 51330. Will do this tomorrow when I'm in the office.

> We also already know what letters 13 other languages would want to use, even
> though we do not have those images ready (I guess), but the best person to
> create those missing images is the person who created the current A/A/B/I/F/K
> ones, I would say.

Our developer/designer time is limited; we did F/K as an example for others to follow, not as a commitment that we would do this for all scripts.
Comment 4 Gerrit Notification Bot 2013-07-19 20:25:53 UTC
Change 74677 had a related patch set uploaded by Jforrester:
Set Malayam to use B/I rather than A/A for toolbar icons

https://gerrit.wikimedia.org/r/74677
Comment 5 Gerrit Notification Bot 2013-07-19 20:27:29 UTC
Change 74677 merged by jenkins-bot:
Set Malayam to use B/I rather than A/A for toolbar icons

https://gerrit.wikimedia.org/r/74677
Comment 6 James Forrester 2013-12-12 19:43:26 UTC
No outstanding requests for these; closing for now as complete.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links