Opened 11 years ago

Last modified 8 months ago

#2724 assigned bug

B_OUTLINE_RESIZE not implemented

Reported by: julun Owned by: nobody
Priority: normal Milestone: R1
Component: Servers/app_server Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description


Attachments (1)

B_OUTLINE_RESIZE-1.patch (14.1 KB) - added by TriEdgeAI 6 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 11 years ago by anevilyak

Component: - GeneralKits/Interface Kit

comment:2 Changed 6 years ago by siarzhuk

Owner: changed from axeld to siarzhuk
Status: newassigned

This ticket was published as GCI 2012 task. ​http://google-melange.appspot.com/gci/task/view/google/gci2012/7984202 So I take ownership of it.

Changed 6 years ago by TriEdgeAI

Attachment: B_OUTLINE_RESIZE-1.patch added

comment:3 Changed 6 years ago by axeld

Component: Kits/Interface KitServers/app_server
Version: R1/pre-alpha1R1/Development

Thanks, TriEdgeAl! The patch looks nicely, I could even only find two minor coding style violations. I haven't tested it, though (since siarzhuk wanted to look at it, too).

Have you by any chance written a test application for this? If so, src/tests/servers/app/ would be a natural place for it -- patch welcome!

comment:4 in reply to:  3 Changed 6 years ago by siarzhuk

Replying to axeld:

Have you by any chance written a test application for this? If so, src/tests/servers/app/ would be a natural place for it -- patch welcome!

As far as I remember, our Magnify application is the one, using B_OUTLINE_RESIZE flag. I have just tested this patch with one of my applications before noticed about Magnify. But the special test application is welcome too! This patch utilizes "non-canonical" way of outline resize - it draws natural window border "teared" off the window. In my personal taste it looks a bit like a buggy behavior so I would like to see some polishing of it. But this is my personal meaning. Another issue - drawing of border on minimizing the frame is hidden under the views of this window - that is loking not nice IMO. As far as I know TriEdge AI wanted to continue working on this feature "in calm" after finishing GCI competition so we can hear from him some more news in the future. :-)

comment:5 Changed 6 years ago by axeld

Thanks siarzhuk! I agree that tearing off the border isn't a nice visual effect; I didn't even get the idea that the patch would do that :-)

Still, I think it's the right way to let the decorator handle this case.

comment:6 Changed 6 years ago by mmadia

Has a Patch: set

comment:7 Changed 6 years ago by mmadia

Is it worthwhile to apply this patch as-is and open new tickets for the tearing issues?

comment:8 in reply to:  7 Changed 6 years ago by siarzhuk

Replying to mmadia:

Is it worthwhile to apply this patch as-is and open new tickets for the tearing issues?

Tearing off is not a problem - the problem is that we have no right solution at all. ;-)

comment:9 Changed 4 years ago by siarzhuk

Owner: changed from siarzhuk to nobody

Those were taken some years ago as potential GCI tasks. Unfortunately no place is available for them in my schedule at this days.

comment:10 Changed 8 months ago by pulkomandy

comment:11 Changed 8 months ago by pulkomandy

Has a Patch: unset
Note: See TracTickets for help on using tickets.