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

Extensions: Show <supersededby> specs in extension table #1472

Merged
merged 1 commit into from
Jan 18, 2025

Conversation

cal0pteryx
Copy link
Member

This adds a button with a popover for all XEP in https://xmpp.org/extensions which have a <supersededby> tag set.

All <spec> elements within the <supersededby> are rendered inside the popover.

If possible, specs are transformed to an HTML link (for XEPs and RFCs).

Depends on xsf/xeps#1418

Example:
image

@iNPUTmice

@iNPUTmice
Copy link
Contributor

Hi,

thank you for that. What I had envision - but not communicated - was to make it way more obvious than that:

Underneath Message Archiving it would say:

Superseded-by: XEP-0313: Message Archive Management

The idea was that if someone is searching for a functionality rather than a specific number they would almost always click on the big, obvious superseded-by link instead of on the XEP itself.

However I’m obviously not the only person that gets to decide this. Maybe we can limit the aforementioned display method to super obvious, clear cut cases. When the XEP is deprecated or deferred and the superseded points to something we can resolve and keep the method from this PR as a fallback for the less obvious ones. (Meaning in your specific example it would probably still show what your example shows because 295 is 'Active'.

And/or we can start by merging this PR and see where this leads us.

@cal0pteryx cal0pteryx force-pushed the extensions-superseded-by branch from 4e71125 to ff25f0a Compare January 18, 2025 16:58
@cal0pteryx
Copy link
Member Author

Thanks! Yes, that's actually a better idea than my first approach. I think it should be obvious now if you look at this screenshot:
image

I added the title as well (at least for XEPs).

@cal0pteryx cal0pteryx force-pushed the extensions-superseded-by branch from ff25f0a to b6cf275 Compare January 18, 2025 23:26
@cal0pteryx cal0pteryx merged commit 8643dc2 into xsf:master Jan 18, 2025
1 check passed
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.

2 participants