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

Icon for Draw files (.*odg) #6416

Closed
merttumer opened this issue Feb 10, 2022 · 11 comments · Fixed by owncloud/owncloud-design-system#2020
Closed

Icon for Draw files (.*odg) #6416

merttumer opened this issue Feb 10, 2022 · 11 comments · Fixed by owncloud/owncloud-design-system#2020
Assignees
Labels

Comments

@merttumer
Copy link

We recently made this pr: owncloud/richdocuments#456 which adds a support for opening .odg files with Collabora Online. But we noticed that there's no icon for the .odg files and it just displayed as regular files.

@pascalwengerter
Copy link
Contributor

Hey @merttumer, thanks for the suggestion! As far as I know CERN (@elizavetaRa @diocas) are planning to add new fileType icons (and perhaps also one for draw documents) to our design system sometime soon, let's check back with them!

Relevant file is in https://github.com/owncloud/core/blob/master/core/img/filetypes/x-office-drawing.svg

@tbsbdr
Copy link
Contributor

tbsbdr commented Feb 23, 2022

@elizavetaRa any news on that 👆 ?

@elizavetaRa
Copy link
Member

@tbsbdr We added some icons here already and some more are planed to come soon. I will create a pr for this.
cernbox/owncloud-design-system@fbd99a2

@pascalwengerter
Copy link
Contributor

@tbsbdr We added some icons here already and some more are planed to come soon. I will create a pr for this. cernbox/owncloud-design-system@fbd99a2

Yes please :) looking forward, and should be easily merged&published!

@pascalwengerter
Copy link
Contributor

@merttumer could you check https://github.com/cernbox/owncloud-design-system/pull/222/files?short_path=96c0611#diff-96c0611a06530f1b881159642cf99d4ce92f33d212670b795e5e0548442bdde6 and see if it suits your needs? (Icon will be added in the CERN fork of the ownCloud design system first and then merged upstream, coming to the web codebase before the next release towards end of next week)

Also, what is the license situation with your icons? We've cleaned up our codebase and are a bit wary to not introduce any offenses again :)

@merttumer
Copy link
Author

merttumer commented Mar 11, 2022

Unfortunately, .odg files are still displayed as black file icon
image

Oh, icon will be added later I see. I see no problem on my side with the icon. I will ask about the license situation :)

@pedropintosilva
Copy link

pedropintosilva commented Mar 11, 2022

Hi there : ) thanks for the heads up @pascalwengerter , (@merttumer has asked me if I could take a look at this and try to help)
So I have checked @elizavetaRa 's patch and they look good ❤️

Generally speaking I do not see any blocker here. Nevertheless and if I might share bit of context:

I would say that for the specific ODG file type I'm afraid resource-type-graphic-fill.svg might be a bit misleading. Even though when working with ODG we have the cool feature of those connectors it is not only for diagrams and there are many other use cases where users/students create drawings or documents such as one page flyer or other promotional materials. Plus it can start to get a bit confusing if we have 3 icons referencing diagrams (resource-type-drawio-fill, resource-type-graphic-fill.svg, and this one)

I would propose to play around with the main elements present in the original mime-type icon while trying to fit it in the current design system : ) here is my quick attempt to do so:

resource-type-draw-fill.svg

What do you think?

@pascalwengerter
Copy link
Contributor

Reopening since it hasn't landed in web yet, also would like to get a positive reply from @pedropintosilva before closing this

@pascalwengerter
Copy link
Contributor

@pedropintosilva @elizavetaRa FYI :)

Screenshot 2022-03-15 at 14-35-03 Personal - ownCloud

@pedropintosilva
Copy link

Seems good @pascalwengerter thanks !

@pascalwengerter
Copy link
Contributor

pascalwengerter commented Oct 11, 2022 via email

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

Successfully merging a pull request may close this issue.

5 participants