Bug 3370

Summary: Suggested change for ConfineTo
Product: xorg Reporter: Egbert Eich <eich>
Component: Server/GeneralAssignee: Peter Hutterer <peter.hutterer>
Status: RESOLVED WONTFIX QA Contact: Xorg Project Team <xorg-team>
Severity: normal    
Priority: high CC: mat, peter.hutterer
Version: git   
Hardware: All   
OS: All   
Whiteboard: 2011BRB_Reviewed
i915 platform: i915 features:
Bug Depends on:    
Bug Blocks: 44202    
Attachments:
Description Flags
Make ConfineToShape jump to the closest postion to the current cursor position inside the shape.
none
confine_to_shape.diff none

Description Egbert Eich 2005-05-23 03:01:13 UTC 
ConfineToShape does not work well: The cursor often times doesn't jum to the
point closest to the current cursor position outside the shape. 
The patch that's going to be attached fixes this. However it exposes some
internals of the RegionRec to the DIX layer. Therefore this code may be debateable.
Comment 1 Egbert Eich 2005-05-23 03:02:39 UTC 
Created attachment 2747 [details] [review]
Make ConfineToShape jump to the closest postion to the current cursor position inside the shape.
Comment 2 Daniel Stone 2007-02-27 01:26:49 UTC 
Sorry about the phenomenal bug spam, guys.  Adding xorg-team@ to the QA contact so bugs don't get lost in future.
Comment 3 Stefan Dirsch 2007-11-04 09:11:26 UTC 
Egbert? See also Novell Bugzilla #62146.
Comment 4 Stefan Dirsch 2008-11-22 13:57:42 UTC 
Patch no longer applies to current git head.
Comment 5 Stefan Dirsch 2009-03-20 04:41:27 UTC 
Created attachment 24080 [details] [review]
confine_to_shape.diff

Patch against xorg-server 1.6. Hope I didn't break it.
Comment 6 Daniel Stone 2009-08-31 18:07:30 UTC 
punting to 7.6.  peter, thoughts?

exposing RegionRec details to the dix is fine, that abstraction is an utter fallacy anyway.
Comment 7 Stefan Dirsch 2010-04-03 03:05:13 UTC 
Giving up ...
Comment 8 Jeremy Huddleston Sequoia 2011-04-11 14:03:38 UTC 
This bug has been around for many releases and does not seem to have a large impact on users.  Because of this, it does not meet the requirements for inclusion in the stable branch (1.10).

Punting to 1.11.
Comment 9 Jeremy Huddleston Sequoia 2011-09-18 01:32:10 UTC 
Peter, please either review this and merge it to your -next/1.12 tree or close it as not to be fixed.
Comment 10 Jeremy Huddleston Sequoia 2011-12-27 21:57:44 UTC 
Deferring to 1.13 as this is functional change
Comment 11 Peter Hutterer 2016-11-28 04:39:53 UTC 
This is a mass change of bugs. Bugs assigned to me that haven't been updated in the last 3 years are closed as WONTFIX, because, well, let's at least be honest about it.

Please do not re-open unless you have a really good reason to do so (e.g. you're fixing it yourself). If it hasn't been fixed in the last 3 years, it probably won't be fixed anytime soon either. Sorry.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.