#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: | ||
Platform: | All |
Description
IMHO, screenshot area in Screenshot app don't need white border, see screenshot :-P
Attachments (2)
Change History (9)
by , 16 years ago
Attachment: | screenshot.png added |
---|
comment:1 by , 16 years ago
Priority: | normal → low |
---|---|
Type: | bug → enhancement |
comment:2 by , 16 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 , 16 years ago
I don't know, but from screenshot I think it would look better without fancy border. What do you think?
by , 16 years ago
Attachment: | screenshot_noborder.png added |
---|
Screenshot showing without fancy border
comment:4 by , 16 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 , 16 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 , 16 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
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.
Will look into it.