-
Notifications
You must be signed in to change notification settings - Fork 1
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
Allow multiple manipulation operations in one link #1
Comments
I took a look at several formats to see how they might be related to this issue, as you can see in Reference Examples below. Just brainstorming, here are a few more ideas to play around with:
Reference ExamplesCommonmark / GF Markdown
Wikitext
Lean versions of HTMLCheck your browser console to see the output from the following formats. SLIM (live)
HAML (live)
Pug (live)
|
One item to consider regarding the pipe: on the Linux command line, that indicates that the result of the previous operation is passed to the next. If that's not true in this case, could it be confusing? If so, we could consider using another symbol like |
Thanks for taking a look 😄 WYSIWYGPersonally, I prefer this option over any Markdown in terms of UI. I think the manipulation operations are just too lengthy for Markdown. However, the issue is development time... The markdown option is so easy to develop and unit test. It's also so easy to integrate into WPs. You can keep the backend storage of descriptions the same and just use jQuery to parse the Markdown. Extending CommonmarkAny extensions to Commonmark should be easy to understand and easy to remember. The latter is harder since some manipulation operations need many parameters and the order of them is crucial. For example, [some text]!!zoomOn 'node1' 'others'!! // easy to remember
[some text]!!toggleHighlight 'node1' 'red' 'others'!! // harder to remember because of the extra parameter. Your ideas
Mistake(?) in CSS and SLIM examples
Just one parameterThinking about everything, the issues really come when there are multiple required parameters. In all of the manipulation functions, there is a resets object as a parameter. For toggleHighlight this is:
I integrated this since it's really needed when you can't do multiple operations in Markdown. However, with multiples enabled you may as well just do (in JS-like syntax):
Where the parameters to the resets are entities that are excluded from resetting. Relating to this issue, if we only allow one parameter per manipulation method then we must use the alternative syntax. |
Yeah, I was wondering about that toggleHighlight syntax. I'll have to think about this a bit. Do Egon or the other researcher you're working with have any preferences? BTW, I ran across another format called edn. Here's an online playground, in case you didn't get enough examples above. 😆 |
Related. Using a WYSIWYG -> markdown editor could solve these issues and also give the possibility for easy state tracking. |
Done in #4 |
Currently, you can only do one manipulation per link. It would be useful for some users to do multiples at once. For example, zooming on a group of nodes and highlighting one of them at the same time.
The syntax for this could be something like
The
|
pipe denotes the seperation of manipulations to perform.What would the RegEx for this be?
The text was updated successfully, but these errors were encountered: