NCAR / geocat-applications

GeoCAT Applications is a community resource inspired by the NCL Applications page.
https://ncar.github.io/geocat-applications/
Apache License 2.0
5 stars 5 forks source link

Reduce Myst Warnings Tags #114

Closed cyschneck closed 2 months ago

cyschneck commented 2 months ago

PR Summary

There is some clutter lingering when building the site locally which can hide when there is a new warning

Related Tickets & Documents

Closes #112

PR Checklist

General

review-notebook-app[bot] commented 2 months ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

github-actions[bot] commented 2 months ago

Meowdy! See your PR preview: 🔍 Git commit SHA: b98b5c22f18f17532648698da9df2115a76b8e6b ✅ Deployment Preview URL: https://NCAR.github.io/geocat-applications/_preview/114

cyschneck commented 2 months ago

It is possible to remove applications/applications.rst:11: WARNING: All children of a 'grid-row' should be 'grid-item' [design.grid] [design.grid]

.. _applications:

.. grid:: 3
    :gutter: 4

    .. grid-item-card:: Dates and Times

        - `Working with Date and Time <applications/date_time/datetime.ipynb>`_

    .. grid-item-card:: Data Analysis

        - `General Applied Math <applications/data_analysis/general_applied_math.ipynb>`_
        - `Calculating Climatologies <applications/data_analysis/climatology.ipynb>`_

    .. grid-item-card:: Plot Types

    .. grid-item-card:: Regridding

But, using the grid-item-card option sets the height of each card to be based on the largest height of the card in the row image Which will get tricky if the card for Data Analysis continues to grow faster than the other cards