Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add placeholder when image is missing from comparison table data #1699

Merged
merged 4 commits into from
May 12, 2022

Conversation

mattseddon
Copy link
Contributor

@mattseddon mattseddon commented May 12, 2022

Related to #1649.

This PR adds a simple placeholder into the comparison table when we have missing data:

Screen Shot 2022-05-12 at 1 20 12 pm

Note: This is a very simple patch and does not work with pinning the revision. It will give us a base to work from.

@mattseddon mattseddon self-assigned this May 12, 2022
@mattseddon mattseddon added bug Something isn't working and removed 🏠 housekeeping labels May 12, 2022
@mattseddon mattseddon marked this pull request as ready for review May 12, 2022 03:49
@mattseddon mattseddon requested a review from a team May 12, 2022 03:53
@shcheklein
Copy link
Member

No image to display -> Change to No plot to show (a bit more future-proof + better to use high-level term I think)

@mattseddon
Copy link
Contributor Author

No image to display -> Change to No plot to show (a bit more future-proof + better to use high-level term I think)

I've updated. This is only a placeholder for the real placeholder while we wait for a design. Should not be around for too long.

@mattseddon mattseddon enabled auto-merge (squash) May 12, 2022 18:29
@codeclimate
Copy link

codeclimate bot commented May 12, 2022

Code Climate has analyzed commit 4e84f3e and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (85% is the threshold).

This pull request will bring the total coverage in the repository to 96.7% (0.0% change).

View more on Code Climate.

@mattseddon mattseddon merged commit 1f933ed into main May 12, 2022
@mattseddon mattseddon deleted the patch-missing-plot-revision branch May 12, 2022 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants