KPAC: Cabin, Kreon default fonts

Bug #1083639 reported by Michael Peters
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Low
Unassigned
2.6
Fix Released
Undecided
Unassigned
2.7
Fix Released
Undecided
Unassigned

Bug Description

In tinkering with KPAC, I've noticed it defaults to using a very nice font, "Cabin", but this font isn't readily available (at least on Windows) without an extra download.

It is, however, available via Google Web Fonts -- http://www.google.com/webfonts/specimen/Cabin

The same case applies for the "Kreon" font which is also used in KPAC.

If we're going to default to using a font most users don't have, perhaps we should leverage Google's API in order to make sure KPAC displays as intended.

Alternatively, we could choose another more "stock" font to ensure users have a consistent experience.

description: updated
summary: - KPAC: Cabin default font
+ KPAC: Cabin, Kreon default fonts
Revision history for this message
Michael Peters (mrpeters) wrote :

Looks like this is as easy as copying the fonts to kpac2...

cp -R /openils/var/web/css/skin/default/kpac/fonts /openils/var/web/css/skin/kpac2/kpac

Revision history for this message
Ben Shum (bshum) wrote :

I think we should add that step to the documentation included for setting up kpac2.

I can help to mock up a branch.

Changed in evergreen:
status: New → In Progress
importance: Undecided → Low
assignee: nobody → Ben Shum (bshum)
milestone: none → 2.3.2
Changed in evergreen:
milestone: 2.3.2 → 2.4.0-alpha
Ben Shum (bshum)
tags: added: documentation
Changed in evergreen:
status: In Progress → Confirmed
assignee: Ben Shum (bshum) → nobody
Ben Shum (bshum)
Changed in evergreen:
milestone: 2.4.0-alpha1 → 2.4.0-beta
Ben Shum (bshum)
Changed in evergreen:
milestone: 2.4.0-beta → 2.4.0-rc
Galen Charlton (gmc)
tags: added: bitesize
Ben Shum (bshum)
Changed in evergreen:
milestone: 2.4.0-rc → none
Kathy Lussier (klussier)
tags: added: bitesize-doc
removed: bitesize
Revision history for this message
Josh Stompro (u-launchpad-stompro-org) wrote :
Changed in evergreen:
status: Confirmed → In Progress
assignee: nobody → Josh Stompro (u-launchpad-stompro-org)
assignee: Josh Stompro (u-launchpad-stompro-org) → nobody
tags: added: pullrequest
Changed in evergreen:
status: In Progress → Confirmed
Revision history for this message
Galen Charlton (gmc) wrote :

Pushed to master, rel_2_7, and rel_2_6; thanks Josh!

Changed in evergreen:
milestone: none → 2.next
no longer affects: evergreen/2.3
no longer affects: evergreen/2.4
Changed in evergreen:
status: Confirmed → Fix Committed
Revision history for this message
Ben Shum (bshum) wrote :

Removing 2.next top level milestone, since this was a bug fix that was repaired in an actual series maintenance release.

Changed in evergreen:
milestone: 2.next → none
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.