Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#3367 closed enhancement (fixed)

[Screenshot] don't need white border

Reported by: diver Owned by: julun
Priority: low Milestone: R1
Component: Applications/Screenshot Version: R1/pre-alpha1
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

IMHO, screenshot area in Screenshot app don't need white border, see screenshot :-P

Attachments (2)

screenshot.png (245.8 KB ) - added by diver 11 years ago.
screenshot_noborder.png (17.1 KB ) - added by stpere 11 years ago.
Screenshot showing without fancy border

Download all attachments as: .zip

Change History (9)

by diver, 11 years ago

Attachment: screenshot.png added

comment:1 by julun, 11 years ago

Priority: normallow
Type: bugenhancement

Will look into it.

comment:2 by stpere, 11 years ago

I made a change to Screenshot application some time ago (hrev30457). There is still the fancy border around the box, but not the white margin. Is it ok for you?

comment:3 by diver, 11 years ago

I don't know, but from screenshot I think it would look better without fancy border. What do you think?

by stpere, 11 years ago

Attachment: screenshot_noborder.png added

Screenshot showing without fancy border

comment:4 by stpere, 11 years ago

I joined a screenshot showing how it would look without the fancy border. You be the judge :) If you think it would look better and nobody opposes, I will commit.

comment:5 by stippi, 11 years ago

I would like the scrollview/text control border (same thing). If nothing else, you can make a view that uses be_control_look to draw the border, or add it to the image displaying view. The BBox border (fancy border) has changed to be 3 pixels wide, and there is a raised inner edge now.

comment:6 by stpere, 11 years ago

Resolution: fixed
Status: newclosed

I committed a fix for this in hrev30593. If it's still not perfect, feel free to reopen this ticket. The new border is definitely more subtle.

comment:7 by diver, 11 years ago

Thanks, the new border is fine :-)

Note: See TracTickets for help on using tickets.