Skip to content

DEV: move RESP information from data to each page #1663

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

Merged
merged 1 commit into from
Jun 4, 2025
Merged

Conversation

dwdougherty
Copy link
Collaborator

@dwdougherty dwdougherty commented Jun 4, 2025

DOC-5299

Moving command return information from data/resp[23]_replies.json to each individual command page using the new multitab shortcode.

Reviewers, please note that there are still a bunch of commands that didn't have their return information in the data files. These will be corrected over time.

@dwdougherty dwdougherty requested review from andy-stark-redis and a team June 4, 2025 15:15
@dwdougherty dwdougherty self-assigned this Jun 4, 2025
@dwdougherty dwdougherty added the dev label Jun 4, 2025
Copy link
Contributor

github-actions bot commented Jun 4, 2025

Staging links:
https://redis.io/docs/staging/DOC-5299/commands/acl-cat
https://redis.io/docs/staging/DOC-5299/commands/acl-deluser
https://redis.io/docs/staging/DOC-5299/commands/acl-dryrun
https://redis.io/docs/staging/DOC-5299/commands/acl-genpass
https://redis.io/docs/staging/DOC-5299/commands/acl-getuser
https://redis.io/docs/staging/DOC-5299/commands/acl-help
https://redis.io/docs/staging/DOC-5299/commands/acl-list
https://redis.io/docs/staging/DOC-5299/commands/acl-load
https://redis.io/docs/staging/DOC-5299/commands/acl-log
https://redis.io/docs/staging/DOC-5299/commands/acl-save
https://redis.io/docs/staging/DOC-5299/commands/acl-setuser
https://redis.io/docs/staging/DOC-5299/commands/acl-users
https://redis.io/docs/staging/DOC-5299/commands/acl-whoami
https://redis.io/docs/staging/DOC-5299/commands/append
https://redis.io/docs/staging/DOC-5299/commands/asking
https://redis.io/docs/staging/DOC-5299/commands/auth
https://redis.io/docs/staging/DOC-5299/commands/bgrewriteaof
https://redis.io/docs/staging/DOC-5299/commands/bgsave
https://redis.io/docs/staging/DOC-5299/commands/bitcount
https://redis.io/docs/staging/DOC-5299/commands/bitfield
https://redis.io/docs/staging/DOC-5299/commands/bitfield_ro
https://redis.io/docs/staging/DOC-5299/commands/bitop
https://redis.io/docs/staging/DOC-5299/commands/bitpos
https://redis.io/docs/staging/DOC-5299/commands/blmove
https://redis.io/docs/staging/DOC-5299/commands/blmpop
https://redis.io/docs/staging/DOC-5299/commands/blpop
https://redis.io/docs/staging/DOC-5299/commands/brpop
https://redis.io/docs/staging/DOC-5299/commands/brpoplpush
https://redis.io/docs/staging/DOC-5299/commands/bzmpop
https://redis.io/docs/staging/DOC-5299/commands/bzpopmax

Copy link
Contributor

@andy-stark-redis andy-stark-redis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dwdougherty I really like this!

Just a thought now I see the tabs "in action", as it were... what do you think about emphasising the notion of the tab a bit more? I'm thinking you could remove the red underline on the selected tab, extend the border around the top of the tab (ie, still underneath the tab title bar), and make the title bar a slightly darker grey. Alternatively, keep the pale grey for the title bar, but add a slight border around it, maybe a little lighter than the border around the tab content and selected tab title. (This is a bit different to the GitHub setup, which has the same colour for all the borders.)

Basically, the white background of the selected tab title should merge into the background of the tab content without the interruption of a line (like the GitHub tabs), but there should be a complete border around the selected tab's title which is the same colour as the border around the tab content.

Anyway, I'll approve even if you don't like the above suggestions. Just thought I'd throw them in :-)

@dwdougherty
Copy link
Collaborator Author

Thanks @andy-stark-redis. I'll look into your UI suggestions separately.

@dwdougherty dwdougherty merged commit efc5b74 into main Jun 4, 2025
5 checks passed
@dwdougherty dwdougherty deleted the DOC-5299 branch June 4, 2025 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants