Page 1 of 1

issue-#145 add scanner option 'maskViews' in DevCommanders

Posted: Tue Feb 14, 2017 10:24 am
by Josef Templ
My users stumbled over an unintended side-effect of BlackBox 1.7 issue #103.
Therefore I have created https://redmine.blackboxframework.org/issues/145 for
removing the side-effect and returning the behavior of BB 1.6.

Here are the details:
BlackBox 1.6 treated a ruler (TextRulers.Ruler) that appeared in front of or as part of the command of a DevCommander as a paragraph character (0EX). With issue #103 of BlackBox 1.7 this behavior has been changed unintentionally and in 1.7 rulers are thereby returned as views (02X). This invalidates commands that contain a ruler or other masked views. The reason for the side effect is that option 'maskViews' was ignored in 1.6 but is honored in 1.7. By adding option 'maskViews' to the scanner used in DevCommanders the behavior of 1.6 can be re-established.

Note: this issue is not about undoing issue #103. It is only about undoing a side-effect
introduced by #103. The cases where it matters are admittedly rare but nevertheless
the problem showed up in a real-life situation and the fix is simple.
So I would propose to consider it.

Here are the diffs:
https://redmine.blackboxframework.org/p ... bd90a41bd0

I have also added the preferences (maskChar, hideable) to the TextRulers docu because this was undocumented so far.

- Josef

Re: issue-#145 add scanner option 'maskViews' in DevCommande

Posted: Tue Feb 14, 2017 3:25 pm
by Ivan Denisov
I think that this suggestion is reasonable.

Re: issue-#145 add scanner option 'maskViews' in DevCommande

Posted: Wed Feb 15, 2017 12:28 pm
by Josef Templ
I have removed the changes in the docu of TextRulers.
Those changes are now handled in issue 146.

- Josef

Re: issue-#145 add scanner option 'maskViews' in DevCommande

Posted: Mon Mar 06, 2017 7:27 pm
by Robert
Should issue 145 be closed?

Re: issue-#145 add scanner option 'maskViews' in DevCommande

Posted: Mon Mar 06, 2017 8:04 pm
by Josef Templ
No, please don't close it.
issue-#145 is ready for voting.

Here is the diff: https://redmine.blackboxframework.org/p ... bd90a41bd0

- Josef