Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 313034 - [Theming] Support horizontal gradients
Summary: [Theming] Support horizontal gradients
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 1.4 M3   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 314763
  Show dependency tree
 
Reported: 2010-05-16 08:15 EDT by Rüdiger Herrmann CLA
Modified: 2010-11-19 12:13 EST (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Rüdiger Herrmann CLA 2010-05-16 08:15:28 EDT
It would be useful to have also horizontal gradients:
  background-image: gradient(
    linear, left, right
    from( red ),
    to( white )
  );
Comment 1 Tim Buschtoens CLA 2010-11-16 05:22:42 EST
We will need that for the new ScrollBar. It would also be great for vertical Sliders, were we should apply it to the thumb and buttons.

As its very easy to also enable diagonal gradients in the process, we should do that. Not because diagonal gradients are very useful (i don't think they are), but because we can then allow any combination of gradient-origin/destination using position-keywords (e.g. bottom right, left top). I think that would be less confusing to the user than some seemingly random restrictions like we have right now.

Btw, the definition given by Rüdiger above is not correct as the origing/destination has to be "x y, x y". It should be:

  background-image: gradient(
    linear, left top, right top,
    from( red ),
    to( white )
  );
Comment 2 Tim Buschtoens CLA 2010-11-19 12:13:08 EST
Fixed in CVS HEAD.

Also updated tests and Slider to use the horizontal gradient if VERTICAL style is used. Thanks to Ivan for the server side implementation. Diagonal gradients are not supported due to time.