-
-
Notifications
You must be signed in to change notification settings - Fork 60
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
Style syntax for related variables #375
base: v1.1
Are you sure you want to change the base?
Style syntax for related variables #375
Conversation
I don't think that's a real problem. Rendering both standard and related names just requires a group in that case, and allowing otherwise would be messy with the multiple="combine" attribute. |
Another problem: if we keep |
I think that would just be empty---I don't think anyone would write that and processors don't need to design any special case around that. |
We could disallow it on styles. But that would be complicated. But maybe it's not necessary. |
@related is much more extensible for the future. |
So, you say better leave it as is? |
Yes. |
I've now also added test syntax. Also, I've changed the wording from |
Sorry I've been MIA. At some point (hopefully soon) I'll get back to these. I note this has conflicts that need resolving, though. |
Just bumping this. Where does it stand? I am assuming, @denismaier, the conflicts will need to be resolved by rebasing this branch on v1.1. See what happens if you do this from the branch dir: git rebase v1.1
git push --force-with-lease |
If and when we do get back this, see note above, and a couple of example fragments would be helpful in the main post. |
Description
This adds syntax for calling related variables.
Overall, I think this should be a simple change. The only shortcoming I see is that it won't be possible to call name variables on different levels from a single
cs:names
element. (Don't know if that's a real problem.)Closes #357
Type of change
Checklist