ycba-cia / blacklight-collections2

5 stars 2 forks source link

Links to objects from publications records in BlackLight #411

Closed yulgit1 closed 1 year ago

yulgit1 commented 1 year ago

Making separate issue for this. Got conflated with https://github.com/ycba-cia/blacklight-collections2/issues/193

yulgit1 commented 1 year ago

@KraigBinkowski @edgartdata @flapka

Please see:

https://collections-test.britishart.yale.edu/catalog/orbis:583000

This is the ref with the biggest number of object references (1223). Unfortunately cannot both render and sort or paginate 1223, as it times-out. So had to put a cap of 250 displayed, you see in that example "Referenced YCBA Works".

Also see: pub_object_count.xlsx

This lists the number of referenced objects for each publication, count descending. The cap will only affect the first 4 of over 4000 with object references. Note there are 38000 reference library total. So hopefully capping 4 of 38000 should be too bad in the grand scheme of things.

Let me know If this is OK to promote to PROD.

KraigBinkowski commented 1 year ago

with those four, 250 makes a loooong list. Do we think it might be ok to cap it even lower? maybe at 150? that would implicate 9 records I think. What do others think?

edgartdata commented 1 year ago

Instead of having a long list of objects or arbitrarily deciding the length of the list, why not have one unique BL link to the group of objects in the book (like what we do with objects in exhibitions) and then let users use the BL facets to narrow down their search further?

yulgit1 commented 1 year ago

That is brilliant. One link to result set based on orbis (ilsnumber) like screenshot:

Screen Shot 2022-10-25 at 4 27 18 PM

https://collections-test.britishart.yale.edu/?utf8=✓&search_field=Fielded&q=ilsnumber_ss%3A583000

OK @KraigBinkowski @flapka?

KraigBinkowski commented 1 year ago

That sounds like a great idea - Kraig

flapka commented 1 year ago

I agree, the one-link approach is better.

yulgit1 commented 1 year ago

Implemented, please see:

https://collections-test.britishart.yale.edu/catalog/orbis:583000 (biggest with 1223 referenced works) https://collections-test.britishart.yale.edu/catalog/orbis:9425121 (example with 1 referenced work) https://collections-test.britishart.yale.edu/catalog/orbis:582683 (example w/o referenced work, (nothing added))

OK to push to prod? @edgartdata @KraigBinkowski @flapka

flapka commented 1 year ago

The functionality is great!

My only quibble or concern is that the link may be too subtle. If others agree, maybe we could do something to make it more prominent. Would it, for example, be worth treating as tabbed content, even though it's so short?

edgartdata commented 1 year ago

I agree with @flapka I think these links should go in a tab on the RBM and Ref Lib BL records.

yulgit1 commented 1 year ago

In tabs now:

https://collections-test.britishart.yale.edu/catalog/orbis:583000 (biggest with 1223 referenced works) https://collections-test.britishart.yale.edu/catalog/orbis:9425121 (example with 1 referenced work) https://collections-test.britishart.yale.edu/catalog/orbis:582683 (example w/o referenced work, (nothing added))

OK to push to prod? @edgartdata @KraigBinkowski @flapka

KraigBinkowski commented 1 year ago

I think this looks great - I would push to prod.

flapka commented 1 year ago

I think the tab works nicely, for prominence.. I suggest tweaking the link language slightly. Perhaps: "View the [x YCBA work(s)] referenced in this item" or something in that vein.

yulgit1 commented 1 year ago

published to prod with language tweak.