Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 338536 - Problem with the refresh of the table : the scrollbar returns to this initial position
Summary: Problem with the refresh of the table : the scrollbar returns to this initial...
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: EMF-Facet (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Gregoire Dupe CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-01 09:20 EST by Vincent Lorenzo CLA
Modified: 2020-05-01 11:26 EDT (History)
2 users (show)

See Also:
gdupe: indigo+

Attachments
Patch for Bug 338536 (6.19 KB, patch)
2011-03-07 05:59 EST, Nicolas Guyomar CLA 		no flags Details | Diff
Patch for Bug 338536 (9.16 KB, patch)
2011-03-09 04:16 EST, Nicolas Guyomar CLA 		gdupe: iplog+
Details | Diff
Show Obsolete (1) 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 Vincent Lorenzo CLA 2011-03-01 09:20:51 EST 
When you modify a property in the table, "sometimes", the refresh of the table does move the scrollbar (the scrollbar returns to this initial position, that is to say, left).

Step to reproduce, using the workbench plugin : 
	1/ Create a table to edit an element (like a class for example)
	2/ Move the scrollbar from its initial position
	3/ Modify the Class visibility
	4/ Click anywhere on the table -> the scrollbar returns to this initial position!
Comment 1 Nicolas Guyomar CLA 2011-03-07 05:59:31 EST 
Created attachment 190535 [details]
Patch for Bug 338536

Hi,

Please find attached a patch which solves this problem. We modified the way the table is redrawn after any modification. The horizontal/vetical scrollbar's states are maintained. The current selection is no longer lost when validating a modification.

Atuhors of this contribution are Nicolas Bros (commiter) and myself:
(a) I, Nicolas Guyomar, wrote 100% of the code I've provided.
(b) I have the right to contribute the code to Eclipse.
(c) I contribute the content under the EPL.

Regards,
Nicolas Guyomar
Comment 2 Nicolas Guyomar CLA 2011-03-09 04:16:37 EST 
Created attachment 190737 [details]
Patch for Bug 338536

Hi,

Please find attached an updated version of this patch, without text indentation differences.

Authors of this contribution are Nicolas Bros (commiter) and myself:
(a) I, Nicolas Guyomar, wrote 100% of the code I've provided.
(b) I have the right to contribute the code to Eclipse.
(c) I contribute the content under the EPL.

Regards,
Nicolas Guyomar
Comment 3 Gregoire Dupe CLA 2011-03-09 04:35:38 EST 
Comment on attachment 190737 [details]
Patch for Bug 338536

Here are a contribution from two employee of Mia-Software, targeting future
Indigo release. The company has signed a Member Commiter Agreement. The
contribution does not need a CQ (see bug 322327).

I've committed this contribution.

Committed revision 348.
Comment 4 Gregoire Dupe CLA 2011-03-09 04:36:20 EST 
This bug can be marked has fixed.
Comment 5 Vincent Lorenzo CLA 2011-03-11 07:22:55 EST 
That works fine. I close the bug.