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

Bug 502409

Summary:	Bidi: Wrong display of server path containing bidi chars
Product:	[ECD] Orion	Reporter:	Moshe WAJNBERG <wajnberg>
Component:	Client	Assignee:	Project Inbox <orion.client-inbox>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	Carolyn_MacLeod, curtis.windatt.public, libingw, Silenio_Quarti, tomerm, wajnberg
Version:	unspecified
Target Milestone:	---
Hardware:	All
OS:	All
See Also:	https://git.eclipse.org/r/82071 https://git.eclipse.org/r/89148 https://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=ddaece485a71d55bc7eae0e3091941e2688b3f37
Whiteboard:

Description Moshe WAJNBERG

2016-09-28 07:45:58 EDT

Select File->New->Link To Server
In the Server Path field type /ABC/DEF where capital letters stand for Hebrew chars

The path is displayed as : /FED/CBA

The right display should be : /CBA/FED

Comment 1 Eclipse Genie

2016-09-28 07:59:23 EDT

New Gerrit change created: https://git.eclipse.org/r/82071

Comment 2 Eclipse Genie

2017-01-19 16:41:35 EST

New Gerrit change created: https://git.eclipse.org/r/89148

Comment 3 Curtis Windatt

2017-02-13 16:03:19 EST

https://dev.eclipse.org/ipzilla/show_bug.cgi?id=12732
This fix is considered a large contribution and requires a CQ. Moshe, can you please comment on this bug stating that you authored 100% of this contribution.

Comment 4 Curtis Windatt

2017-02-13 16:06:10 EST

Also Moshe, there are two Gerrit changes marked on this bug, please specify which one you want reviewed and integrated.

Comment 5 Moshe WAJNBERG

2017-02-13 16:13:14 EST

Yes I certify I authored 100% of this contribution.
Also this change https://git.eclipse.org/r/89148 should be integrated

Comment 6 Curtis Windatt

2017-02-15 16:52:35 EST

(In reply to Moshe WAJNBERG from comment #5)
> Yes I certify I authored 100% of this contribution.
> Also this change https://git.eclipse.org/r/89148 should be integrated

As you are an IBM employee, the contribution falls under Figure 2 of [1].  This bypasses the 'large contribution' CQ requirement.

Libing, you are free to merge the change.

[1] https://eclipse.org/legal/EclipseLegalProcessPoster.pdf

Comment 7 Eclipse Genie

2017-03-01 11:51:25 EST

Gerrit change https://git.eclipse.org/r/89148 was merged to [master].
Commit: http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=ddaece485a71d55bc7eae0e3091941e2688b3f37

Comment 8 Carolyn MacLeod

2017-03-02 15:49:04 EST

Please see: Bug 513012 - Can't see default text in dark themes