Skip to content

@W-23748894 fix(webhooks): align event name with method pill in URL area - #102

Merged
alexpmule merged 1 commit into
masterfrom
fix/webhook-url-alignment
Aug 26, 2026
Merged

alexpmule merged 1 commit into
masterfrom
fix/webhook-url-alignment

Conversation

@alexpmule

Copy link
Copy Markdown
Contributor

What

Aligns the webhook event name with the method pill in the operation URL area.

Why

For OAS 3.1/3.2 webhooks, api-url renders the event name inside .url-server-value instead of a URL/server. That class carries margin-top: 8px, which exists to separate it from a preceding .url-channel-value ("Channel") line in the non-HTTP (AsyncAPI/protocol) case, where .url-value stacks two blocks. A webhook has no "Channel" line, so .url-server-value is the sole child and the 8px margin just pushes the event name out of vertical alignment with the method pill.

Fix

Zero the top margin only when .url-server-value is the first child of .url-value:

.url-value > .url-server-value:first-child { margin-top: 0; }
  • Webhook: .url-server-value is the sole/first child → margin removed → event name aligns with the method pill.
  • Non-HTTP ("Channel" + "Server" stacked): .url-server-value follows .url-channel-value → not :first-child → keeps the 8px separation.
  • Plain HTTP URL and AsyncAPI server renders: don't use .url-server-value as a first child → unaffected.

Version

Bumps to 5.2.34.

Refs

The webhook event name renders inside .url-server-value, whose 8px top margin exists to separate it from a preceding "Channel" line in the non-HTTP case. For a webhook there is no Channel line, so .url-server-value is the sole child and the margin just pushes the event name out of line with the method pill. Zero the margin only when .url-server-value is the first child; the non-HTTP "Channel + Server" stack is unaffected.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@alexpmule
alexpmule merged commit a04807a into master Aug 26, 2026
4 checks passed
This was referenced Aug 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant