Skip to content

Replace some \textit#8081

Open
eisenwave wants to merge 5 commits intocplusplus:mainfrom
eisenwave:replace-textit
Open

Replace some \textit#8081
eisenwave wants to merge 5 commits intocplusplus:mainfrom
eisenwave:replace-textit

Conversation

@eisenwave
Copy link
Member

Related to #2076

I'm not sure if it's worth dismantling all uses of \textit. There are ~150 left. This is just picking some low-hanging fruits in wording that might still be visited from time to time.

@eisenwave eisenwave added the P2-Bug Presentational errors and omissions label Nov 6, 2025
@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 7, 2025

Thanks! Could you have a look also in text.tex, that also contains three instances of "see \ref{...}" in a \textit.

@eisenwave
Copy link
Member Author

I'll put it on my list of things to do once the repo is public and I can make updates to my PRs without things exploding.

@eisenwave eisenwave added the changes requested Changes to the wording or approach have been requested and not yet applied. label Nov 10, 2025
@jensmaurer
Copy link
Member

@eisenwave , the (technical) blockers are gone here, I think.

@eisenwave eisenwave requested a review from tkoeppe February 25, 2026 17:48
@eisenwave eisenwave removed the changes requested Changes to the wording or approach have been requested and not yet applied. label Feb 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P2-Bug Presentational errors and omissions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants