ZeligsoftDev / CX4CBDDS

CX4CBDDS component modeling and generation tool
Apache License 2.0
8 stars 5 forks source link

Change modeling tool to only allow plain text when copying into documentation fields #425

Closed emammoser closed 1 year ago

emammoser commented 2 years ago

Issue and tracking information

It's very common to copy and paste documentation worked in a confluence(Wiki website that NGC uses) page. This introduces significant invisible whitespace changes that are a lot of work to change. Best way to fix it is modifying the XML of the model itself. Update the modeling tool to only allow plain text when copying and pasting into documentation fields.

Developer's time Estimated effort to fix (hours):

Developer's Actual time spent on fix (hours)

Issue reporter to provide a detailed description of the issue in the space below

ysroh commented 1 year ago

@j26151 Looks like you can paste the text only if you select the "Source" button before pasting it. Can this be a workaround?

emammoser commented 1 year ago

I think this works as a work around. Is there any way to configure the panel to be "Source" by default?

ysroh commented 1 year ago

Unfortunately, that is not possible if the rich text editor is enabled. We enabled the rich text editor for Issue #252 but turning it off would be going back to the text-only comments editor.

emammoser commented 1 year ago

Okay, we will talk a bit on our side to see what we want to do. Thank you for your response

emammoser commented 1 year ago

Two things. It appears on the head of papyrus, that the Rich text editor errors out pretty quickly. Here is a screenshot of the exception: image I can get this pretty easily by opening up papyrus, navigating to somewhere that we have a comment, seeing the rich text editor open up, then clicking around on other items/comments/the same comment and the rich text editor will eventually just disappear for good. I think we would be fine with disabling the rich text editor now as it looks like the regular text editor is working and allowing us to scroll around inside of it and edit lines that extend outside of the box.

ysroh commented 1 year ago

I didn't notice this in Windows but was able to reproduce it in Linux. I will enable the text editor and test it.

ysroh commented 1 year ago

Raised the following bug report to Papyrus: https://bugs.eclipse.org/bugs/show_bug.cgi?id=580867

emammoser commented 1 year ago

In the meantime, I think it would solve this ticket if we uncheck the use rich text editor option in the preferences and delivered that to our users.

ysroh commented 1 year ago

Sounds good. I am making that change now.

emammoser commented 1 year ago

Could we have the tag for v2.2.0 updated to include this fix? @J17359

ysroh commented 1 year ago

Yes. I will be doing that.

ysroh commented 1 year ago

verified.