Bug 490949 - Themed StyledText scroll bar is missing context menu
Summary: Themed StyledText scroll bar is missing context menu
Status: RESOLVED WONTFIX
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.6   Edit
Hardware: All All
: P3 major (vote)
Target Milestone: 4.6 M7   Edit
Assignee: Fabio Zadrozny CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 430278 491032
  Show dependency tree
 
Reported: 2016-04-02 04:29 EDT by Dani Megert CLA
Modified: 2016-04-13 05:36 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dani Megert CLA 2016-04-02 04:29:30 EDT
Themed StyledText scroll bar is missing context menu.

Instead it unexpectedly opens the editor's context menu.
Comment 1 Fabio Zadrozny CLA 2016-04-04 06:18:39 EDT
Note: will provide a pull request for this tomorrow.
Comment 2 Eclipse Genie CLA 2016-04-05 10:19:46 EDT
New Gerrit change created: https://git.eclipse.org/r/69922
Comment 3 Fabio Zadrozny CLA 2016-04-05 12:13:43 EDT
This pull request should fix it: https://git.eclipse.org/r/69922 (so, the editor context menu is no longer changed when requested over the scroll bar).
Comment 5 Lars Vogel CLA 2016-04-11 17:01:35 EDT
Looks fine to me. Thanks Fabio. 

To be honest, I didn't even know that we have a context menu on the StyledText scroll bar. Dani might want to retest this one.
Comment 6 Dani Megert CLA 2016-04-13 05:31:51 EDT
(In reply to Lars Vogel from comment #5)
> Looks fine to me. Thanks Fabio. 
> 
> To be honest, I didn't even know that we have a context menu on the
> StyledText scroll bar.

That's not specific to StyledText. Most scroll bars on Windows have a context menu.


> Dani might want to retest this one.

I assume you don't plan to provide the context menu, hence marking as WONTFIX.
Comment 7 Dani Megert CLA 2016-04-13 05:36:24 EDT
See also bug 491577.