Skip to content

Conversation

oesteban
Copy link
Member

@oesteban oesteban commented May 3, 2019

Builds upon #344, #346, #349, #351, and #352

  • Resizing of reportlets: now reportlets can be either static or dynamic. Static reportlets are embedded in <img> tags, which are easier to resize (CSS works just fine). Conversely, dynamic reportlets are embedded in <object> tags, which are more finicky and require proper settings of height, width, viewBox and preserveAspectRatio. Animations do not work within <img> tags.
  • Better ordering of functional subreports.
  • Improved subtitles and other minor amends.

  - [x] Resizing of reportlets: now reportlets can be either static or
    dynamic. Static reportlets are embedded in ``<img>`` tags, which are
    easier to resize (CSS works just fine). Conversely, dynamic reportlets
    are embedded in ``<object>`` tags, which are more finicky and
    require proper settings of ``height``, ``width``, ``viewBox`` and
    ``preserveAspectRatio``. Animations do not work within ``<img>`` tags.
  - [x] Better ordering of functional subreports.
  - [x] Improved subtitles and other minor amends.
@oesteban oesteban requested a review from rwblair May 3, 2019 01:36
oesteban added a commit to oesteban/fmriprep that referenced this pull request May 3, 2019
@oesteban oesteban merged commit 093c08f into nipreps:master May 3, 2019
@oesteban oesteban deleted the fix/compcor-reportlet-naming branch May 3, 2019 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants