Opened 10 years ago

Closed 10 years ago

Last modified 10 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 10 years ago.
screenshot_noborder.png (17.1 KB) - added by stpere 10 years ago.
Screenshot showing without fancy border

Download all attachments as: .zip

Change History (9)

Changed 10 years ago by diver

Attachment: screenshot.png added

comment:1 Changed 10 years ago by julun

Priority: normallow
Type: bugenhancement

Will look into it.

comment:2 Changed 10 years ago by stpere

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 Changed 10 years ago by diver

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

Changed 10 years ago by stpere

Attachment: screenshot_noborder.png added

Screenshot showing without fancy border

comment:4 Changed 10 years ago by stpere

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 Changed 10 years ago by stippi

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 Changed 10 years ago by stpere

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 Changed 10 years ago by diver

Thanks, the new border is fine :-)

Note: See TracTickets for help on using tickets.