Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Files/Documents #79

Closed
ericfennis opened this issue Sep 21, 2020 · 9 comments · Fixed by #131
Closed

Files/Documents #79

ericfennis opened this issue Sep 21, 2020 · 9 comments · Fixed by #131

Comments

@ericfennis
Copy link
Member

Similar: #579

Icon Request

  • Icon name:
    files
  • Use case:
    For multiple files/documents
@ghost
Copy link

ghost commented Oct 19, 2020

Just an idea
image
100%
files

@moeenio
Copy link
Contributor

moeenio commented Oct 19, 2020

This more feels like Copy to me

@ericfennis
Copy link
Member Author

I agree, I think that's because the line is at the bottom.
I like the approach, maybe it work better if we turn it around. First the file and then line in the background (top-right), so it looks more like a pile of papers.

@ghost
Copy link

ghost commented Oct 19, 2020

Like this?
image
100%
files(1)

@ericfennis
Copy link
Member Author

hmm, not sure about this either 🤔
Maybe the first is better, vscode uses that one as well.

@ericfennis
Copy link
Member Author

@ghost
Copy link

ghost commented Oct 21, 2020

what the hell have I done
image

@ericfennis
Copy link
Member Author

hmm, oke I after looking at the other options I think your first one is maybe the best choice, I can't think of better options. I agree I maybe looks a bit like 'copy files' as wel, but maybe we can embrace that with a tag: "copy files". So people can use it for both 🤷‍♂️?

@ghost
Copy link

ghost commented Oct 25, 2020

uhm well... You opened the issue so if you're satisfied just take it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants