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

mod: syntax highlighting: Improve syntax highlighting (deployment + keywords) #425

Merged
merged 6 commits into from
Oct 12, 2021

Conversation

The-Lum
Copy link
Contributor

@The-Lum The-Lum commented Jul 27, 2021

1/ Syntax highlighting: adding missing deployment keywords, in order to improve syntax highlighting:

@startuml
agent agent
artifact artifact
card card
circle circle
collections collections
file file
hexagon hexagon
label label
person person
queue queue
stack stack
storage storage
@enduml

2/ Adding keywords:

  • mainframe (on type 'line begin keywords')
  • map ('line begin keywords')
  • kill ('whole line keywords")
  • end merge ('whole line keywords")
  • repeat (adding on 'line begin keywords')
  • backward ('line begin keywords')

This PR was inspired by this:

syntax highlighting: adding missing deployment keywords
in order to improve syntax highlighting:
```
@startuml
agent agent
artifact artifact
card card
circle circle
collections collections
file file
hexagon hexagon
label label
person person
queue queue
stack stack
storage storage
@enduml
```
Adding keywords:
- mainframe (on type 'line begin keywords')
- map ('line begin keywords')
- kill ('whole line keywords")
- end merge ('whole line keywords")
- repeat (adding on 'line begin keywords')
- backward ('line begin keywords')
@The-Lum
Copy link
Contributor Author

The-Lum commented Jul 27, 2021

FYI, in order to Improve syntax highlighting, and create separate little tasks/commits, I create this project:

Improve preprocessing keyword
See: https://plantuml.com/preprocessing

Remains to be done:
[] foreach in/endfor
[] Affectation
Add `diamond` keyword.
Why `abstract class` is not good ?

```
@startuml
abstract        abstract
abstract class  "abstract class"
diamond         diamond
@enduml
```
After some tests, in order to manage `abstract class`:
- `abstract class` must be declared before `abstract`!
Standardize all `...|abstract\s+class|abstract|...` occurrence.
@qjebbs qjebbs merged commit 873482a into qjebbs:master Oct 12, 2021
@qjebbs
Copy link
Owner

qjebbs commented Oct 12, 2021

Thanks!

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