Bug 515036 - Selected day of month in due date picker is not visible
Summary: Selected day of month in due date picker is not visible
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 4.9   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 4.9 M2   Edit
Assignee: Xi Yan CLA
QA Contact:
URL:
Whiteboard:
Keywords: triaged
Depends on:
Blocks:
 
Reported: 2017-04-10 10:47 EDT by Mat Booth CLA
Modified: 2018-08-03 09:41 EDT (History)
2 users (show)

See Also:

Attachments
Illustration of the problem (109.35 KB, image/png)
2017-04-10 10:47 EDT, Mat Booth CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mat Booth CLA 2017-04-10 10:47:02 EDT 
Created attachment 267729 [details]
Illustration of the problem

See attached screenshot. When creating a new local task, using the due-date picker widget, the currently selected day-of-month is not visible.

OS: linux (RHEL 7.3)
ARCH: x86_64
WS: gtk (3.14.13)

This is in a fresh install of the latest Oxygen I-build, build ID: 4.7.0.I20170409-2000

With Mylyn 3.22.0 installed from the Oxygen update site.
Comment 1 Roland Grunberg CLA 2018-07-03 10:57:22 EDT 
Reproduced with Photon 4.8 and Mylyn 3.24.0.
Comment 2 Roland Grunberg CLA 2018-07-04 13:14:12 EDT 
This is reproducible in the DateTime widget itself, which is actually used here.

Simply open ControlExample from SWT examples. Go to the DateTime tab and select SWT.CALENDAR from the right. Now change the background colour to red. Note how the selected day is white but there's no background for the selection itself.
Comment 3 Eric Williams CLA 2018-07-04 13:38:42 EDT 
This is a question of adding the selection background when setting the DateTime's background color. Xi, please investigate.
Comment 4 Eclipse Genie CLA 2018-07-04 15:04:49 EDT 
New Gerrit change created: https://git.eclipse.org/r/125558
Comment 6 Eric Williams CLA 2018-07-05 09:50:45 EDT 
(In reply to Eclipse Genie from comment #5)
> Gerrit change https://git.eclipse.org/r/125558 was merged to [master].
> Commit:
> http://git.eclipse.org/c/platform/eclipse.platform.swt.git/commit/
> ?id=92026a06e12d118b31b3cea7fd4d86c877f3acbb

Patch is in master, thanks Xi.