The current modal size is relatively small and has to be increased if you want to display additional fields, or even a description. However, CTools doesn't detect it when a modal size is specified that is actually larger than the screen. So, if we increase modal size (currently 250x170), we should also add code checking that to ensure we don't end up with glitches on smaller displays.

CommentFileSizeAuthor
#2 2870749-2--increase_modal_size.patch1.43 KBdrunken monkey
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drunken monkey created an issue. See original summary.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
CreditAttribution: drunken monkey for Vizala commented
Status: Active » Needs review
FileSize
2870749-2--increase_modal_size.patch1.43 KB

This patch would implement that functionality.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
CreditAttribution: drunken monkey for Vizala commented
Status: Needs review » Fixed

Committed.

  • drunken monkey committed 255df4d on 7.x-1.x 
    Issue #2870749 by drunken monkey: Increased the modal size but made it...

Status: Fixed » Closed (fixed)

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