Would be nice to be able to include fonts in features and this should also be useful for creating a unified set of font tags.

CommentFileSizeAuthor
#12 1080038-12_exportables-for-7.x-2.0-beta1.patch11.6 KBpatcon
#9 exportables_changes.patch3.86 KBBarisW
#9 plugins.zip3.46 KBBarisW
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bforchhammer’s picture

bforchhammer CreditAttribution: bforchhammer commented

I agree, would be great if we could export font settings! *subscribing*

drupal_was_my_past’s picture

drupal_was_my_past CreditAttribution: drupal_was_my_past commented

subscribe

sreynen’s picture

sreynen CreditAttribution: sreynen commented
Version: 7.x-1.x-dev » 7.x-2.x-dev

This should happen in 7.x-2.x now.

BarisW’s picture

BarisW CreditAttribution: BarisW commented

doubleclick..

BarisW’s picture

BarisW CreditAttribution: BarisW commented

We probably have to use strings as identifiers in the database instead of serials (like Rules does). If we keep using numbers as ID's, it will be hard to keep track of fonts across multiple environments. We could use identifiers like: [provider]_[font_path]

eg: google_fonts_api_ubuntu_mono

sreynen’s picture

sreynen CreditAttribution: sreynen commented

I was thinking the URL field could work as a UUID across environments, since that's already specific to the provider-font combination.

My vague plan without looking at this much was to use Chaos Tools exportables functionality, just because that seems to be how most modules are handling exportables.

BarisW’s picture

BarisW CreditAttribution: BarisW commented
Component: Code » Code (general)
Assigned: Unassigned » BarisW
sreynen’s picture

sreynen CreditAttribution: sreynen commented

Hey Baris,

You mentioned in IRC that the Chaos Tools exports you did don't show up in Views. If you can post the code you have for the exports, I'd like to look for a solution to that problem.

BarisW’s picture

BarisW CreditAttribution: BarisW commented
FileSize
plugins.zip3.46 KB
exportables_changes.patch3.86 KB

Sure! See attached files. The plugins.zip should be extracted in the fontyourface_ui folder.

BarisW’s picture

BarisW CreditAttribution: BarisW commented

Posted a support request here as well: #1315086: How to combine Views and Ctools Exportables?

BarisW’s picture

BarisW CreditAttribution: BarisW commented
Status: Active » Fixed

Fixed and committed to 7.x-2.x-dev.

The solution was not to use ctools (as it has some limitation, see #1315086: How to combine Views and Ctools Exportables? but to use custom feature hooks, like described in http://treehouseagency.com/blog/thomas-wysocki/2010/10/28/making-your-mo...

patcon’s picture

patcon CreditAttribution: patcon commented
FileSize
1080038-12_exportables-for-7.x-2.0-beta1.patch11.6 KB

This is awesome guys! For anyone who needs a patch against 7.x-2.0-beta1 for a makefile, here it is :)

BarisW, you are a hero

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

davideads’s picture

davideads CreditAttribution: davideads commented

Will there be a 6.x backport of this functionality?

sreynen’s picture

sreynen CreditAttribution: sreynen commented
Version: 7.x-2.x-dev » 6.x-2.x-dev
Status: Closed (fixed) » Patch (to be ported)

We're not doing much more than maintenance on D6 at this point. That said, I'll add this to the long list of "to be ported" issues and if someone makes a patch, I'll review it.

davideads’s picture

davideads CreditAttribution: davideads commented

Thanks. I'm in my final week of work on this project, so I probably won't be able to roll a patch myself, but if I have a few minutes I'll give it a whack.

Neslee Canil Pinto’s picture

Neslee Canil Pinto
he/him
Primary language English
CreditAttribution: Neslee Canil Pinto at Specbee commented
Issue summary: View changes
Status: Patch (to be ported) » Closed (outdated)