Opened 6 years ago

Closed 6 years ago

#8830 closed enhancement (fixed)

Automatic export of localized badge graphics

Reported by: dcf Owned by: dcf
Priority: Low Milestone:
Component: Archived/Flashproxy Version:
Severity: Keywords:
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Badge localizations are stored as different layers in badge.xcf. You open badge.xcf in Gimp, made your chosen layer visible, then export to the PNG you want. This would be better if automated from the command line.

This should be driven by the makefile and Gimp batch mode, maybe with an auxiliary shell script.

LANGS = de en ru
all: $(addprefix badge-,$(addsuffix .png,$(LANGS))
badge-%.png: badge.xcf
        gimp --batch something

Child Tickets

Attachments (3)

0001-Overlay-localized-text-on-badge-graphic-with-CSS.patch (7.4 KB) - added by arlolra 6 years ago.
8830#2-screenshot.png (450 bytes) - added by dcf 6 years ago.
How the badge looks with a text overlay and a minimum font size
0001-Automatic-export-of-localized-badge-graphics.patch (3.3 KB) - added by arlolra 6 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 6 years ago by arlolra

Since the translations are already present in JS for the alt tag, wouldn't it be better to remove the text from the .png and just overlay the image in CSS? This would make it easier to add translations as well. You wouldn't have to edit anything in Gimp. I'd be happy to supply a patch.

comment:2 Changed 6 years ago by arlolra

Status: newneeds_review

I've added a patch for the idea I described above.

comment:3 Changed 6 years ago by dcf

Status: needs_reviewnew

Naw man, I like your idea for maintainability, but as long as we're doing the silly pixel art thing we should do it right. Here's a screenshot of what your patch looks like on my Firefox, where I configure a minimum font size. Even when the text fits, with antialiasing and everything it doesn't fit the established web badge aesthetic.

Changed 6 years ago by dcf

Attachment: 8830#2-screenshot.png added

How the badge looks with a text overlay and a minimum font size

comment:4 Changed 6 years ago by arlolra

Status: newneeds_review

Fair enough. I attached the patch you wanted this time. Enjoy.

comment:5 Changed 6 years ago by dcf

Resolution: fixed
Status: needs_reviewclosed

Thanks, this was really great work.

Note: See TracTickets for help on using tickets.