endoflife.date icon indicating copy to clipboard operation
endoflife.date copied to clipboard

[Feature] Better icon support

Open captn3m0 opened this issue 2 years ago • 9 comments

This creates a new release filter to help parse the releases in code. Using Liquid for generating the actual product page is good for simple prose, but gets too complicated for the remaining part. This can help us move to a simpler codebase, where the complicated parts are handled by the code, and the Liquid parts have very little logic.

This would help us make progress on getting API to parity with the HTML (The goal being to have a Release filter that actually parses the page into something both the API and HTML can consume)

captn3m0 avatar Jun 16 '22 10:06 captn3m0

Tried to figure out our performance issues while working on this, and filed https://github.com/just-the-docs/just-the-docs/issues/863

Turns out adding categories will speed up our builds.

captn3m0 avatar Jun 16 '22 10:06 captn3m0

Added logos to products that do not have logos such as Almalinux, cfengine and others

Evernow avatar Jun 16 '22 10:06 Evernow

A small suggestion , couldnt we make this one

icon:
  simpleicons: blender

change something like this :

icon:
  iconProvider: "simpleicons"
  iconName: "blender"

or maybe if you wish we can combine provider and iconname into one string using @ in its name

icon: "blender@simpleicons"

This approach is more cleaner what do you thing ? @captn3m0 @BiNZGi

usta avatar Jun 16 '22 13:06 usta

I'm okay with the last one if everyone likes it. It's not too confusing for contributors?

My main intent with the PR was to remove the "NA" and automatic detection of icons, and instead force everything to be explicit.

captn3m0 avatar Jun 16 '22 14:06 captn3m0

yeah i think it is really cleaner and more understandable for event a new contributor using name@provider syntax and it will make our .md files much shorter and error prune

usta avatar Jun 16 '22 14:06 usta

I also suggest similar approach for our dates within #1334

usta avatar Jun 18 '22 14:06 usta

The files would be shorter indeed but I like to separate details because it is less confusing. The name and the provider of an icon are two separate things and I like it better to have separate. I vote for this proposal:

icon:
  iconProvider: "simpleicons"
  iconName: "blender"

BiNZGi avatar Jun 22 '22 14:06 BiNZGi

How about keeping it slightly terse:

icon:
  provider: simpleicons
  name: blender

captn3m0 avatar Jun 23 '22 07:06 captn3m0

Any news about this pr/mr? @captn3m0

usta avatar Sep 13 '22 00:09 usta

Decided against this, icon styles weren't fitting. Better to contribute to SimpleIcons instead.

Closing

captn3m0 avatar Nov 24 '22 14:11 captn3m0