cljdoc

serioga 2023-03-10T16:30:12.503169Z

hm, I think that I saw value of the :added metadata in documentation. But I don't see this now. Am I glitching?

lread 2023-03-11T14:03:26.850209Z

Also @corasaurus-hex is very often helpful with ui design trade-offs. She might, if she has time, chime in (absolutely no pressure Cora, I expect you are busy these days).

Cora (she/her) 2023-03-11T17:43:54.260919Z

people come to docs to see what they can use and how they can use it and so deprecated is more important to know about than added. that said, it's not obvious how to get the right version of docs for a library (you click/tap the version at the top to pick different versions) and so looking at the latest docs it may be useful to see added. you won't also see "removed", though, so that's of limited utility

Cora (she/her) 2023-03-11T17:45:01.681859Z

I think well-done UI can show added and not feel cluttered. we could also improve how versions are selected

Cora (she/her) 2023-03-11T17:47:38.729639Z

there will always be a tension between what the cljdoc team thinks people need to see and what authors would like them to see, unfortunately. I like the idea of showing added, I sometimes find it helpful context, like a git blame but in the code

lread 2023-03-11T18:44:08.637789Z

thanks @corasaurus-hex, much appreciated

👍🏻 1
martinklepsch 2023-03-12T20:35:27.003639Z

we’ve been talking about semantic diffs (based on analysis data) and effectively using that to see changes between library versions.

martinklepsch 2023-03-12T20:36:51.665209Z

don’t see anything wrong with supporting some representation of :added though

lread 2023-03-12T20:44:25.384109Z

coolio, thanks @martinklepsch

lread 2023-03-10T16:49:45.367309Z

Hi @serioga, cljdoc does not currently show added metadata, it does show deprecated metadata. It treats it as a boolean. Example:

serioga 2023-03-10T16:52:14.316189Z

But I have a memory of light grey ADDED label under function name...

lread 2023-03-10T17:00:18.337019Z

Not sure, maybe it did once? Could you be thinking of some codox generated docs? https://weavejester.github.io/medley/medley.core.html#var-deep-merge:

serioga 2023-03-10T17:01:39.659439Z

Ah, maybe another documentation system...

lread 2023-03-10T17:04:00.415609Z

Could be! My brain is foggy on this too. If cljdoc did start to show :added it would have to show the value, and for consistency, we'd like also show the value of :deprecated.

💯 1
lread 2023-03-10T17:05:09.715029Z

I'm pretty sure we collect and save :added during API analysis.

lread 2023-03-10T17:07:30.265559Z

Given cljdoc's sparse design aesthetic, I'm guessing :added was omitted in the spirit avoiding visual clutter. But if it is considered generally useful info we should consider including it.

serioga 2023-03-10T17:11:37.340119Z

If author adds this information then author think that this info is useful :-)

lread 2023-03-10T17:21:07.853769Z

Yep, all trade-offs ui minimalism vs info. For example the fact that a function is deprecated is more important than what version it was deprecated, hence cljdoc just showing "deprecated" (if I understand the design choice correctly). And deprecation is typically much more important to a user of a library than what version a var was added. I personally don't have a preference on this one. If @martinklepsch has a free moment, he might chime in.

serioga 2023-03-10T17:31:53.428859Z

I feel like you mean that ADDED conflicts with DEPRECATED but I don't understand why :-)

serioga 2023-03-10T17:34:16.478959Z

Deprecated is important, there is no standard for it's value afaik (but I would not mind see non-boolean value as well). Also it would be cool to show deprecated vars as striked-out in the sidebar's navigation (including namespaces).

lread 2023-03-10T17:45:09.163499Z

> I feel like you mean that ADDED conflicts with DEPRECATED but I don't understand why :-) Nope, just trying to convey that DEPRECATED is probably more important information than ADDED for users of a library. > Also it would be cool to show deprecated vars as striked-out in the sidebar's navigation (including namespaces) Ya, I too was thinking some indication of deprecation would be nice in index views.