FlowCrypt / flowcrypt-browser

FlowCrypt Browser extension for Chrome and Firefox
https://flowcrypt.com
Other
372 stars 46 forks source link

visual design: attachments (send + receive) #51

Closed tomholub closed 8 years ago

tomholub commented 8 years ago

I just added a way to send encrypted attachments (for now only text files) and also receive them.

I used a very basic design I'll show in the screenshots, and a better design would be awesome :)

Sending attachments

Receiving attachments

2016-03-09-110909_851x830_scrot

Please post screenshots/photoshop deisgns here first (as a reply) and then we put it into code later

thanks!

mondo989 commented 8 years ago

For Sending attachments:

I agree we should somewhat close to Gmail, but still just a little different. We can use the x and put the lock icon in front as they are effective. The lock will be nice reassurance for the user to know that their attachment is encrypted. The attach icon is a little larger.

screen shot 2016-03-08 at 10 07 11 pm

For Receiving attachments:

Sweet, we get an iframe! We can show the name within the logo and also show the download icon, which is something that the user has a habit of seeing.

recieve

Let me know any feedback

tomholub commented 8 years ago

sending - I like it

receiving - consider this scenario - some files might be encrypted and some not, so it might look like this: image

So I think it would make sense in this case to make it a bit more consistent with gmail. Maybe throw away the "text/plain" and then have the name of the file in a footer similar as the other one. Maybe the thing on the bottom right should be green? Let me know

mondo989 commented 8 years ago

For receiving, I agree.

Can we change the properties in the bottom right corner?

screen shot 2016-03-09 at 12 07 16 am
tomholub commented 8 years ago

Looks great. Yes whatever you need to do :) you might need to draw your own assets for the little guy in the corner. I just copied it from gmail, but badly.

Tom James Holub http://holub.me/

On Wed, Mar 9, 2016 at 4:09 PM, Armando notifications@github.com wrote:

For receiving, I agree.

Can we change the properties in the bottom right corner?

[image: screen shot 2016-03-09 at 12 07 16 am] https://cloud.githubusercontent.com/assets/3431871/13628972/10b93724-e58b-11e5-8b1c-00d04ed2af4a.png

— Reply to this email directly or view it on GitHub https://github.com/tomholub/cryptup-chrome/issues/51#issuecomment-194172247 .

tomholub commented 8 years ago

To implement this, you can unlink <link rel="stylesheet" href="/lib/fine_uploader/fine-uploader-new.css" /> and add the necessary styles into gmail_elements.css. For receiving, right now that is also at the end of that file, the lines body.attachment ...

mondo989 commented 8 years ago

Here's how the attached text files look right now.

screen shot 2016-03-11 at 1 27 34 am screen shot 2016-03-11 at 1 25 31 am
mondo989 commented 8 years ago

Maybe we can have the file size as how gmail has it?

screen shot 2016-03-11 at 1 46 32 am

Also want to align it left...

mondo989 commented 8 years ago

Current extra feature.... Whenever the user clicks on the anything that's not the X, the element becomes content editable and the x goes a little wild.

screen shot 2016-03-11 at 2 07 35 am

Sorry for all the spam

tomholub commented 8 years ago

I'll disable the click, np. And definitely, let's add size, just say (1K) everywhere and I'll fix it after. Looks good!

Tom James Holub http://holub.me/

On Fri, Mar 11, 2016 at 6:09 PM, Armando notifications@github.com wrote:

Current extra feature.... Whenever the user clicks on the anything that's not the X, the element becomes content editable and the x goes a little wild.

[image: screen shot 2016-03-11 at 2 07 35 am] https://cloud.githubusercontent.com/assets/3431871/13699293/4bc1e70e-e72e-11e5-8448-647be06596eb.png

Sorry for all the spam

— Reply to this email directly or view it on GitHub https://github.com/tomholub/cryptup-chrome/issues/51#issuecomment-195305854 .

mondo989 commented 8 years ago

Cool!

I've pushed updates...

One thing to note is that after 3 text files the new ones get pushed below the email's body input, it basically just disappears... Notice on the third attached †ext file, it kind of starts to fall under...

screen shot 2016-03-11 at 4 36 58 pm
tomholub commented 8 years ago

Yes I was thinking about this.. I will fix that for now, the danger is that these files will stack all the way up and then obstruct the written text. I have an idea how to fix it, that we can explore later, for now I will allow 10 attachments per email or so.

Tom James Holub http://holub.me/

On Sat, Mar 12, 2016 at 8:38 AM, Armando notifications@github.com wrote:

Cool!

I've pushed updates...

One thing to note is that after 3 text files the new ones get pushed below the email's body input, it basically just disappears... Notice on the third attached †ext file, it kind of starts to fall under... [image: screen shot 2016-03-11 at 4 36 58 pm] https://cloud.githubusercontent.com/assets/3431871/13719242/9a19b872-e7a7-11e5-8823-d2b10766cb05.png

— Reply to this email directly or view it on GitHub https://github.com/tomholub/cryptup-chrome/issues/51#issuecomment-195617134 .

tomholub commented 8 years ago

not sure if I broke it or not, but it's showing like this now:

2016-03-12-110727_688x715_scrot

tomholub commented 8 years ago

correction, the X got fixed somehow. Also fixed more than 3 attachments.

The only thing left for now are the little file iframes, thanks!

mondo989 commented 8 years ago

Glad it fixed. The scroll bar is a bit ugly on yours, so I added some css to hopefully hide the scrollbar. It still goes to content editable on click.

tomholub commented 8 years ago

Good work!