README: show feature gifs in a 2x2 grid#609
Merged
Merged
Conversation
Comment on lines
+56
to
+57
| <td><img src="gifs/autocorrect.gif" alt="Cotabby autocorrect demo" width="460" height="260" /></td> | ||
| <td><img src="gifs/macros.gif" alt="Cotabby inline macros demo" width="460" height="260" /></td> |
Contributor
There was a problem hiding this comment.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds the new autocorrect and macros gifs to the README header, and lays all four gifs out in an evenly sized 2x2 grid so they line up cleanly on GitHub.
Validation
Visual only. Rendered locally; all four gifs resolve and the grid is symmetric.
Linked issues
None.
Greptile Summary
Restructures the README demo section from two side-by-side
<img>tags into an HTML<table>-based 2×2 grid, and introduces two new gif references (autocorrect.gif,macros.gif) alongside the existingslack.gifandimessage.gif.gifs/autocorrect.gifandgifs/macros.gifare referenced but not committed to the repository, so those two cells will show as broken images until the assets are added.Confidence Score: 3/5
Two of the four referenced gif assets are not present in the repository, so merging as-is will display broken images in the README demo grid.
The grid layout itself is correct, but
gifs/autocorrect.gifandgifs/macros.gifare absent from the repo. Visitors to the project page will see broken image placeholders in the bottom row until those files are committed.README.md — the two new gif paths need their corresponding asset files added to the
gifs/directory.Important Files Changed
<img>tags with an HTML<table>2×2 grid; adds two new gif references (autocorrect.gif,macros.gif) that do not exist in the repo.Flowchart
%%{init: {'theme': 'neutral'}}%% flowchart TD A[README Demo Section] --> B[Before: two img side-by-side] A --> C[After: table 2x2 grid] C --> D[Row 1: slack.gif checkmark] C --> E[Row 1: imessage.gif checkmark] C --> F[Row 2: autocorrect.gif missing] C --> G[Row 2: macros.gif missing]Reviews (1): Last reviewed commit: "README: show feature gifs in a 2x2 grid ..." | Re-trigger Greptile