hub icon indicating copy to clipboard operation
hub copied to clipboard

feat: Show last updated badge for alternate plugin variants

Open ReubenFrankel opened this issue 1 year ago • 9 comments

Closes #1293

image

Happy to add "Last updated" back, but thought it looked cleaner without - or we can do something on hover as suggested in #1293.

ReubenFrankel avatar Mar 22 '24 01:03 ReubenFrankel

Deploy Preview for meltano-hub ready!

Name Link
Latest commit d6f0f3a2db91c746e87ee3428ce8029f2908446f
Latest deploy log https://app.netlify.com/sites/meltano-hub/deploys/667a7cfe151d360008b91bff
Deploy Preview https://deploy-preview-1719--meltano-hub.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Mar 22 '24 01:03 netlify[bot]

We could add the Airbyte badge back once https://github.com/badges/shields/pull/10041 is released (i.e. last commit for connector path in the repo).

ReubenFrankel avatar Mar 27 '24 16:03 ReubenFrankel

I think @tayloramurphy had some concerns about the limited real-estate in the current plugin pages so I kinda wanna let him comment

edgarrmondragon avatar Mar 27 '24 16:03 edgarrmondragon

@edgarrmondragon @pnadolny13 can we get the build for this working so I can see a preview? Hard to compare with just a screenshot.

tayloramurphy avatar Apr 04 '24 02:04 tayloramurphy

Thanks for fixing the build @ReubenFrankel!

Things do seem a bit crowded on mobile:

Screenshot_2024-04-04-08-13-08-969_com.duckduckgo.mobile.android-edit.jpg

cc @tayloramurphy

https://deploy-preview-1719--meltano-hub.netlify.app/extractors/tap-tiktok

edgarrmondragon avatar Apr 04 '24 14:04 edgarrmondragon

@edgarrmondragon Ah yeah, that doesn't look amazing... I forgot about mobile.

ReubenFrankel avatar Apr 04 '24 15:04 ReubenFrankel

I don't love this. The badging was an attempt to indicate overall quality - but I get why you'd just want to see the latest quick ASAP.

I like the proposal of a hover or tap to view some of the data - maybe based on the badge? @pnadolny13 what do you think?

tayloramurphy avatar Apr 04 '24 21:04 tayloramurphy

I like the proposal of a hover or tap to view some of the data - maybe based on the badge?

@ReubenFrankel @tayloramurphy @edgarrmondragon I agree that seeing all of the variant metadata next to each other helps a lot when comparing but were running out of space. The idea of a hover sounds nice. If we had a Compare variants tool tip that would be ideal. Like put all the variants next to each other in a tabular view to compare across the board that pops up when you click. Idk how challenging these types of things are though.

pnadolny13 avatar Apr 17 '24 15:04 pnadolny13

I'll have a look into a tooltip implementation when I find some more time to work on this. :slightly_smiling_face:

ReubenFrankel avatar Apr 18 '24 02:04 ReubenFrankel

Current state:

image

If we had a Compare variants tool tip that would be ideal. Like put all the variants next to each other in a tabular view to compare across the board that pops up when you click.

Wasn't quite sure how to interpret this, but sounds like it would benefit from a design first - I imagine you would want to compare across more than just last updated date, such as quality and built with the SDK (i.e. existing badges)?

ReubenFrankel avatar Jun 03 '24 14:06 ReubenFrankel

I deployed a preview to https://deploy-preview-1779--meltano-hub.netlify.app/extractors/tap-purecloud but the hovers don't seem to work? @pnadolny13 any idea?

tayloramurphy avatar Jun 08 '24 13:06 tayloramurphy

@tayloramurphy I think that deploy was targeting https://github.com/meltano/hub/commit/e78acb7ca5f406e9fe7c1751495d8842c4f40b5a, which I accidentally pushed to this branch the other day (hence previous force-push back).

https://github.com/meltano/hub/actions/runs/9428997282

ReubenFrankel avatar Jun 08 '24 14:06 ReubenFrankel

the tooltip badges look a lot better 🙌

edgarrmondragon avatar Jun 25 '24 10:06 edgarrmondragon

@ReubenFrankel this is awesome! Thanks for contributing this.

pnadolny13 avatar Jun 25 '24 22:06 pnadolny13