Skip to content

Initialized fonts dom element ASAP instead of waiting for appinit #8525

Closed
wants to merge 1 commit into from

2 participants

@MiguelCastillo
Adobe Systems Incorporated member
@redmunds

@MiguelCastillo @peterflynn The bug referenced in the description (#8524) is High Priority, Milestone 0.42, & FBNC. Is this still a valid fix? If so, do we want to get this in for Release 0.42?

@MiguelCastillo
Adobe Systems Incorporated member

@redmunds This PR addresses the underlying issue that is exposed when adding line-heights. Since line-height is no longer in the code, the issue is no longer visible. However, the code in this PR is "more correct" but I don't think is worth adding it to 42 as there is no apparent side effect we can find. I think we can merge this in 43.

@MiguelCastillo
Adobe Systems Incorporated member

@redmunds The code in this PR is so outdated, I couldn't get a nice clean merge going. I am going to create a new one...

@MiguelCastillo MiguelCastillo deleted the unknown repository branch Aug 4, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.