Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 544617

Summary: [formatting] Operators not displayed correctly on profile page
Product: [Eclipse Project] JDT Reporter: Noopur Gupta <noopur_gupta>
Component: UIAssignee: Noopur Gupta <noopur_gupta>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: daniel_megert, mateusz.matela, sarika.sinha
Version: 4.11Flags: sarika.sinha: review+
Target Milestone: 4.11 RC1   
Hardware: All   
OS: All   
See Also: https://git.eclipse.org/r/137512
https://git.eclipse.org/r/137513
https://git.eclipse.org/c/jdt/eclipse.jdt.ui.git/commit/?id=82f4e5e641fe9e1f4161bab38df6c86da200aa04
https://git.eclipse.org/c/www.eclipse.org/eclipse/news.git/commit/?id=dc02907d4b214a0624cc0f100ea567597c2e6af0
Whiteboard:
Bug Depends on: 543475    
Bug Blocks:    
Attachments:
Description Flags
Screenshot none

Description Noopur Gupta CLA 2019-02-20 05:08:27 EST
Created attachment 277630 [details]
Screenshot

I20190219-1800

Under Binary Operators on the formatter profile, the operators are not displayed correctly. 

See attached screenshot. 
- In first two, the first operator is missing.
- In last two, '&' is shown as a logical operator which is wrong.
Comment 1 Noopur Gupta CLA 2019-02-20 05:23:41 EST
Mateusz, please fix this and also update the N&N screenshot.

For M3, this should be fixed today before 18:00 Eastern Time. Otherwise, it should be released after M3 is declared for RC1 with another +1 from the project lead.

N&N can be updated anytime. Also, N&N says, "The new Binary expressions subsection..." whereas it should be "Binary operators".
Comment 2 Noopur Gupta CLA 2019-02-20 08:09:02 EST
(In reply to Noopur Gupta from comment #1)
> Mateusz, please fix this and also update the N&N screenshot.
> 
> For M3, this should be fixed today before 18:00 Eastern Time. Otherwise, it
> should be released after M3 is declared for RC1 with another +1 from the
> project lead.
> 
> N&N can be updated anytime. 

> Also, N&N says, "The new Binary expressions
> subsection..." whereas it should be "Binary operators".
I have updated the N&N text.
Comment 3 Eclipse Genie CLA 2019-02-25 04:32:19 EST
New Gerrit change created: https://git.eclipse.org/r/137512
Comment 4 Eclipse Genie CLA 2019-02-25 04:32:41 EST
New Gerrit change created: https://git.eclipse.org/r/137513
Comment 5 Sarika Sinha CLA 2019-02-25 06:32:10 EST
Looks good for RC1.
Comment 8 Noopur Gupta CLA 2019-02-25 07:17:28 EST
Thanks, Sarika.

Released the fix and updated the N&N screenshot for RC1.
Comment 9 Mateusz Matela CLA 2019-02-25 18:46:57 EST
I thought the N&N screenshot better presented the concept when some checkboxes were not checked. Also, your version has a different font size, maybe it would be better to keep it consistent at least between the formatter screenshots.

Shall I upload my version of the screenshot?
Comment 10 Noopur Gupta CLA 2019-02-26 00:41:54 EST
(In reply to Mateusz Matela from comment #9)
> Shall I upload my version of the screenshot?
Yes, please update.
Comment 11 Mateusz Matela CLA 2019-02-26 17:25:23 EST
(In reply to Noopur Gupta from comment #10)
> Yes, please update.

Done.
Comment 12 Noopur Gupta CLA 2019-02-28 03:15:52 EST
Verified in I20190226-1800.