printdotio / printio-ios-sdk

SDK that enables the printing of any photo, from any source, onto any product!
Other
21 stars 11 forks source link

CQA: Mousepad update icons to show black grid #231

Closed briantemple closed 9 years ago

briantemple commented 10 years ago

Unlike UI for other products, mousepad background is solid black. Due to the background color, it is impossible to see drag areas for product.

When images are added the images have black borders around/between them and image corners overlap black rounded corners on bottom.

img_0916 img_0917

AustenB commented 10 years ago

@briantemple what are your suggestions

briantemple commented 10 years ago

@AustenB For consistency, probably give the empty mousepad target a white background like framed prints so the drop areas can be seen. Fix the layout issues when images are applied to match what the product looks like. Or does the mousepad have black bars around the images? I haven't actually seen a printed multi-image mousepad, and the examples only show single image mousepads, so maybe I have made an improper assumption about this screen.

AustenB commented 10 years ago

Collages have black borders yes. We plan to update and change this company wide so that images are given "rounded" containers to match the outside.

But for now it is what it looks like. (Mousepads look bad in white)

On Wed, Sep 10, 2014 at 11:23 AM, Brian Temple notifications@github.com wrote:

@AustenB https://github.com/AustenB For consistency, probably give the empty mousepad target a white background like framed prints so the drop areas can be seen. Fix the layout issues when images are applied to match what the product looks like. Or does the mousepad have black bars around the images? I haven't actually seen a printed multi-image mousepad, and the examples only show single image mousepads, so maybe I have made an improper assumption about this screen.

— Reply to this email directly or view it on GitHub https://github.com/printdotio/printio-ios-sdk/issues/231#issuecomment-55131847 .

Austen Bernstein Founder & CEO print.io | Hellopics Breakout Commerce Group

briantemple commented 10 years ago

The photo layout/design screen should be updated then, as the current previews do not show this:

img_0961

AustenB commented 10 years ago

Valid point. This is a data issue for web & mobile.

On Thu, Sep 11, 2014 at 11:40 AM, Brian Temple notifications@github.com wrote:

The photo layout/design screen should be updated then, as the current previews do not show this:

[image: img_0961] https://cloud.githubusercontent.com/assets/303257/4236785/b3dbc652-39c9-11e4-98f2-6d7427f4981e.PNG

— Reply to this email directly or view it on GitHub https://github.com/printdotio/printio-ios-sdk/issues/231#issuecomment-55282935 .

Austen Bernstein Founder & CEO print.io | Hellopics Breakout Commerce Group

briantemple commented 10 years ago

@micahasmith For reference, here is what it currently looks like on web:

screen shot 2014-09-11 at 10 55 41 am

AustenB commented 10 years ago

@briantemple new mousepad overlay updated

briantemple commented 10 years ago

@AustenB I don't see any difference in c6f65ab.

AustenB commented 10 years ago

I am positive it is fixed on staging. Live cache hasn't been cleared. When its cleared than this will auto use right one

briantemple commented 10 years ago

@AustenB Well, that explains it -- our builds/testflight only use production on iOS. Please let us know when it is ready to test on production.

briantemple commented 10 years ago

@AustenB Preview icons still have grey background today on b236fe40e15e10d05ff6e5251805de455764415c.

AustenB commented 10 years ago

We updated the overlay - if we need to add black in between the grids we can do that also. I am ticketing. Will be done tomorrow.