Hi

The default short preview text in admin->config->user-interface/fontyourface is not length limited.

Using the Google fonts API and the text 'Sleepy wizards eat dogs for breakfast before quidditch' creates a bit of a mess in
appearance->fontyourface->browse->google_fonts_api

The text box for each font doesn't extend to accommodate the text.

As a quick fix - not necessarily the correct fix - I've edited the title in the fontyourface-iu module to include a hint about the max desirable length of chars.

Am patching it.

The pictures show the state of the display when using the release and the dev version of @font-your-face respectively.

CommentFileSizeAuthor
#1 fontyourface-Default-text-2142109-2.patch773 byteslesleyb
display.error_.2.png176.86 KBlesleyb
display.error_.1.png198.01 KBlesleyb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lesleyb’s picture

lesleyb CreditAttribution: lesleyb commented
FileSize
fontyourface-Default-text-2142109-2.patch773 bytes
lesleyb’s picture

lesleyb CreditAttribution: lesleyb commented

Added a patch

lesleyb’s picture

lesleyb CreditAttribution: lesleyb commented
Status: Active » Needs review
BarisW’s picture

BarisW CreditAttribution: BarisW commented

'Sleepy' fits, so shouldn't we set it to 6?
And why not use #maxlength here?

Neslee Canil Pinto’s picture

Neslee Canil Pinto
he/him
Primary language English
CreditAttribution: Neslee Canil Pinto at Specbee commented
Status: Needs review » Closed (outdated)