Spacing issue with popups that include embedded images about docassemble

[jhpyle]

This is more of a Bootstrap issue than a docassemble issue.

In the Bootstrap CSS, popovers have a hard-coded max-width of 276px. I don't know that I want to go down the road of trying to override the decisions of the people at Twitter who created the popover system. The hard-coded width of 276px might be optimal in a smartphone-centric environment.

You can include your image with a width specifier, e.g., [FILE sample_document.png, 100%], and it will fit nicely in the popover. Images without a width have a max-width of 300px by default in docassemble.

[nonprofittechy]

Adding ,100% does solve the issue. Thanks.

On Mon, Feb 26, 2018 at 4:15 PM Jonathan Pyle ***@***.***> wrote: This is more of a Bootstrap issue than a docassemble issue. In the Bootstrap CSS, popovers have a hard-coded max-width of 276px. I don't know that I want to go down the road of trying to override the decisions of the people at Twitter who created the popover system. The hard-coded width of 276px might be optimal in a smartphone-centric environment. You can include your image with a width specifier, e.g., [FILE sample_document.png, 100%], and it will fit nicely in the popover. Images without a width have a max-width of 300px by default in docassemble. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#50 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AHSpyeh9G-iowHwAy0BrCOV87PhMinHdks5tYx69gaJpZM4ST50o> .

-- Quinten Steenhuis https://twitter.com/QSteenhuis github.com/nonprofittechy nonprofittechy.com

[jhpyle]

The code has been changed so that 100% width is the default for images included inside popovers.