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

Add inter as an alternative for sect (Proposal 14, part I) #17

Merged
merged 5 commits into from
Feb 3, 2025

Conversation

MDLC01
Copy link
Collaborator

@MDLC01 MDLC01 commented Dec 12, 2024

This PR implements the first part of Proposal 14 (iteration 1) from the Symbol Proposals document. It adds inter as an alternative for sect. Eventually, sect should be deprecated, and later removed. For more context, see the discussion on Discord.

For now, there is no infrastructure to deprecate symbols in Codex, nor is there a way to deprecate a constant in Typst. This makes it impossible to deprecate sect immediately. However, since sect is a commonly used symbol, I believe the sooner we introduce its new name, the better.

Since this PR assumes the future removal of a symbol (sect) and a variant (integral.sect), and even though this was already discussed on Discord, cc. @laurmaedje.

@MDLC01
Copy link
Collaborator Author

MDLC01 commented Dec 14, 2024

We should probably wait for #19 to be closed before merging this PR.

@T0mstone
Copy link
Collaborator

I'll mention #27 here, since this PR introduces a duplicated symbol.

@laurmaedje
Copy link
Member

With #19 merged, we can proceed with the deprecation of sect.

@MDLC01
Copy link
Collaborator Author

MDLC01 commented Feb 3, 2025

I marked sect as deprecated. There is no way to mark the integral.sect variant as deprecated with the current infrastructure, but I think this is fine as it is not a widely use symbol. Also, this does not remove it anyway, so nothing will break for now.

src/modules/sym.txt Outdated Show resolved Hide resolved
src/modules/sym.txt Outdated Show resolved Hide resolved
MDLC01 and others added 2 commits February 3, 2025 23:21
No warning though, as the current infrastructure does not support it.
@MDLC01 MDLC01 merged commit 6e7634f into typst:main Feb 3, 2025
1 check passed
@MDLC01 MDLC01 deleted the proposal-14 branch February 3, 2025 22:54
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.

3 participants