Repository navigation
DOC-7128 icons-2: convert rs/7.22, 7.8, and 7.4 inline icons - #4193
Merged
Merged
Conversation
Contributor
Contributor
Contributor
🧠 Redis MemoryFound 7 related items from repository history:
Memory updated at a812125 |
Runs build/migrate_image_shortcodes.py --inline-icons over 54 files in the rs/7.22/, rs/7.8/, and rs/7.4/ snapshots, converting 70 #no-click icon shortcodes to inline Markdown images sized by the CSS rule from icons-1. Stacked on icons-1, which adds the hook class and the CSS. Verified against an icons-1 build under the production baseURL: all 70 render with the same src, no width, and class img-inline, with no other image changes. A scripted browser pass over all 54 affected pages found every icon at 1.5x its text size and every single-line block (64) at its normal line height. Ticket: DOC-7128 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
andy-stark-redis
force-pushed
the
DOC-7128-icons-2
branch
from
October 2, 2026 15:06
1b295dd to
a812125
Compare
EliShteinman
added a commit
to EliShteinman/docs
that referenced
this pull request
Oct 3, 2026
Converts the rs/7.22, 7.8, and 7.4 inline icons to Markdown images. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
2 participants
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
Converts the remaining
#no-clickUI icon shortcodes, in thers/7.22/,rs/7.8/, andrs/7.4/snapshots, to inline Markdown images sized by the CSS from #4192 (merged). Part 2 of 2 for icons in DOC-7128.Counts: 70 icons converted in 54 files (18 in
7.22/, 28 in7.8/, and 24 in7.4/). No#no-clickicon shortcodes are left in these snapshots. The same two classes of image as in #4192 are left alone: inline screenshots without#no-click, and theinfo-icon.pngimages.Visible changes: the same as #4192. Icons scale to 1.5× their text, and lines or table rows holding an icon lose the extra prose-margin padding.
Verification
/docs/latest/): both succeeded (19,664 pages), and the hook reported no missing images.src, nowidth, and classimg-inline, with 0 unexpected changes anywhere. No image without#no-clickgained the class.#no-clickshortcodes left in the 54 files. In every changed file, the text with all images stripped is byte-identical before and after.🤖 Generated with Claude Code
Note
Low Risk
Documentation-only markup migration with no product or API changes; rendering depends on #4192’s CSS/hook already in the stack.
Overview
Part 2 of DOC-7128: finishes migrating Redis Enterprise Software docs in the
rs/7.22/,rs/7.8/, andrs/7.4/version snapshots from Hugo{{< image ... #no-click >}}shortcodes to inline Markdown images ().70 icons across 54 files are updated—cluster node actions, database overflow menus, edit/delete/checkmark buttons, warning status icons, and similar UI affordances in procedural steps. Prose and steps are unchanged; only the markup for those inline icons changes. Icons pick up the
img-inlineclass from the Goldmark render hook (from stacked PR #4192) instead of shortcodewidth/classattributes.No
#no-clickimage shortcodes remain in these three snapshots. Larger screenshots and non-#no-clickimages are untouched.Reviewed by Cursor Bugbot for commit 1b295dd. Bugbot is set up for automated code reviews on this repo. Configure here.