Global: support font loading mechanism

[ianmcburnie]

Skin will set Market Sans by default in DS6 global module (EDIT: not true anymore, see thread)

To enable a font loader mechanism, let's add a Lasso flag that will bypass this setting. (EDIT: not true anymore, see thread)

https://github.com/eBay/ebay-font

[senthilp]

@ianmcburnie Can we have the font loader as the default and use a flag as an override? I already checked with design team and they are also fine with it. Going forward we are going to advocate everyone to use the font loader if they want the Market Sans font

[ianmcburnie]

@senthilp Yes, we can do it that way around if you prefer. I presume the font loader will be JS framework agnostic? Also this ticket has a dependency on the ebay-font-loader being moved over to open source.

For simple static web sites that cannot use a font loader, I think we will have to set Market San as the default though, but that will be via the CDN url.

[senthilp]

Although the JavaScript of font loader is independent of any framework, the way we are exposing it is through a Marko tag. Since Marko also is open sourced, will it still be a concern?

I am fine with Market San being used explicitly in CDN url, as that is needed for static sites.

[ianmcburnie]

Hmm, I think this is the original reason I wanted Market Sans set by default, without requiring configuration, so that we remain true to our "decoupled" philosophy. But maybe I'm taking that philosophy too far here? I guess we are already coupled to Lasso to a large extent for app usage.

I think the bottom line though is that we have to do what's best for our own eBay use case. So I will defer that decision to you :-)

[senthilp]

We should keep that philosophy. Meanwhile, we will try to document it as such, that even without the Marko dependency we will be able to use the font loader. Will keep you posted.

[ianmcburnie]

Blocked while we try and figure out if this goes against the design system guidance of Market Sans for 16px and over only:

eBay/ebay-font#8

[ianmcburnie]

Okay, I'm going to set this as the default font-family for apps:

font-family: "Helvetica Neue", Helvetica, Arial, Roboto, sans-serif;

Meaning ebay-font will be required in order to load and set Market Sans in apps.

The CDN bundle used by Raptor and static sites (that cannot leverage ebay-font) will be:

font-family: "Market Sans", "Helvetica Neue", Helvetica, Arial, Roboto, sans-serif;

I don't think we need any Lasso flag now, if we do it this way around.

PR coming soon.