On Thu, Dec 13, 2018 at 11:54 AM Thomas Singer <[email protected]> wrote:
> I'd prefer it to be optional, because the StyledText might be used in > situations where this behavior might be unexpected. IMHO the best > approach would be to make it not part of the official StyledText (if > possible), but instead implement it to a separate class without the > StyledText class knowing about it. If this is not possible, please > implement it that way that it needs to be enabled for each StyledText > instance individually. I think we can discuss that kind of things once we have a 1st version of the patch. Depending on the implementation, there could be various ways to approach the enabled/disabled issue, it's too early to find out the best one IMO and we should first get a patch that enables/disables with an explicit flag (with something like StyledText.enableButton3Scroll(button)) and then we'll be more able to discuss the default value. Do you happen to maintain one of such StyledText that would be negatively impacted by this feature?
_______________________________________________ platform-dev mailing list [email protected] To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/platform-dev
