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

Improve formatting of definitions in the Push Notifications section #1415

Merged
merged 2 commits into from
Jan 29, 2023

Conversation

HarHarLinks
Copy link
Contributor

@HarHarLinks HarHarLinks commented Jan 25, 2023

Signed-off-by: Kim Brose 2803622+HarHarLinks@users.noreply.github.com

I thought I skipped a beat when I tried to read this https://spec.matrix.org/unstable/client-server-api/#push-notifications:
image

The same idea might apply to the rest of the spec, but I don't have time to search for it.

Preview: https://pr1415--matrix-spec-previews.netlify.app

Signed-off-by: Kim Brose <2803622+HarHarLinks@users.noreply.github.com>
@HarHarLinks HarHarLinks requested a review from a team as a code owner January 25, 2023 17:30
Copy link
Member

@uhoreg uhoreg left a comment

Choose a reason for hiding this comment

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

Good catch. Looks like this was originally a definition list, but it got lost in the conversion. Could you try switching it use the Markdown definition list markup. There's an example a bit later, under the Push Rules > Actions section (line ~190 of push.md).

changelogs/internal/newsfragments/1415.clarification Outdated Show resolved Hide resolved
Signed-off-by: HarHarLinks <2803622+HarHarLinks@users.noreply.github.com>
@HarHarLinks HarHarLinks changed the title Improve formatting of push.md Improve formatting of definitions in the Push Notifications section Jan 26, 2023
@uhoreg uhoreg merged commit d4586eb into matrix-org:main Jan 29, 2023
@uhoreg
Copy link
Member

uhoreg commented Jan 29, 2023

Thanks!

clokep pushed a commit to clokep/matrix-spec that referenced this pull request May 3, 2023
…atrix-org#1415)

Signed-off-by: Kim Brose <2803622+HarHarLinks@users.noreply.github.com>
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.

None yet

2 participants