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

Formatter bug in Eclipse Vorto Editor #857

Closed
inno-steffg opened this issue Feb 28, 2018 · 0 comments · Fixed by #877
Closed

Formatter bug in Eclipse Vorto Editor #857

inno-steffg opened this issue Feb 28, 2018 · 0 comments · Fixed by #877

Comments

@inno-steffg
Copy link

As soon as I am dragging a dependency into a functionblock or datatype, my sources are formatted automatically.

Unfortunately, the documentation parts of property definitions are moved to the start of the next property definition. Human readers will then easily be confused and think these documentation belongs to that other property.

Example

before

        optional busy_to_clear as Delay with {
            measurementUnit : StandardUnits.ms
        }
        "how long will it need after activity was measured, that the measurement can toggle to no activity the earliest"
        
        optional clear_to_busy as Delay with {

after auto formating

        optional busy_to_clear as Delay with {
            measurementUnit : StandardUnits.ms
        }

        "how long will it need after activity was measured, that the measurement can toggle to no activity the earliest"       
        optional clear_to_busy as Delay with {
Ebolon added a commit to Ebolon/vorto that referenced this issue Apr 12, 2018
… add two after.

Signed-off-by: Simon Pizonka <simon@pizonka.de>
Ebolon added a commit to Ebolon/vorto that referenced this issue Apr 12, 2018
Ebolon added a commit to Ebolon/vorto that referenced this issue Apr 12, 2018
Signed-off-by: Simon Pizonka <simon@pizonka.de>
aedelmann pushed a commit that referenced this issue Apr 13, 2018
* Fixes #857. Removes line wraps before property description and add two after.

Signed-off-by: Simon Pizonka <simon@pizonka.de>

* Revert "Fixes #857. Removes line wraps before property description and add two after."

This reverts commit d8e1e06.

* Fixes #857 - Removes line wraps before property description

Signed-off-by: Simon Pizonka <simon@pizonka.de>
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 a pull request may close this issue.

1 participant