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

Some more thoughts about references <-> related videos #63

Closed
Tracked by #25
shiffman opened this issue Feb 9, 2022 · 9 comments
Closed
Tracked by #25

Some more thoughts about references <-> related videos #63

shiffman opened this issue Feb 9, 2022 · 9 comments
Assignees

Comments

@shiffman
Copy link
Member

shiffman commented Feb 9, 2022

I realize my comments keep leading in the direction of more work for me, but in looking at the additional flexible tabs (that i"m using for "REFERENCES" and "VIDEOS" right now), I wonder if these might feel less "blank" to me, if everything included three elements:

  1. Emoji icon (Custom one / Coding Train character?)
  2. Title (linked to URL)
  3. Short description

I imagine this relates to the discussion around #57.

e.g.

    {
      "title": "References",
      "links": [
        {
          "icon": "🔢",
          "title": "Fisher-Yates Shuffle"
          "url": "https://bost.ocks.org/mike/shuffle/"
          "description": "The Fisher-Yates shuffle is an algorithm for generating a random permutation of a finite sequence",
        },
        {
          "icon": "✏️"
          "title": "copy()"
          "url": "https://p5js.org/reference/#/p5/copy"
          "description:": "The full details about copy() are on the p5.js reference.",
        }
      ]
    },

Here's what it looks like me putting all this metadata just in the title

Screen Shot 2022-02-09 at 11 19 30 AM

@fdoflorenzano
Copy link
Collaborator

I like this idea! It shouldn't be hard to implement too.

@shiffman
Copy link
Member Author

shiffman commented Feb 9, 2022

Great! Would it makes sense for me to start adding some sample content with the above JSON structure in the branch I'm working in? #60

@fdoflorenzano
Copy link
Collaborator

Yes! From that I can do, yet another, branch and test it out :)

@shiffman
Copy link
Member Author

I've added the sample metadata to the slide puzzle challenge! a8a0a49. I'd be ok with doing away with "author" if that is helpful, it can always be in the description. One question is whether the descriptions can accept any html tags, like if I wanted to link to something additional?

@fdoflorenzano
Copy link
Collaborator

One question is whether the descriptions can accept any html tags, like if I wanted to link to something additional?

It can be done! Are you just thinking about anchor elements?

@fdoflorenzano
Copy link
Collaborator

Also, just to make sure, are you thinking of removing the "author" property in favor of "description", right? @shiffman

@shiffman
Copy link
Member Author

It can be done! Are you just thinking about anchor elements?

That's all that comes to mind now yes!

Also, just to make sure, are you thinking of removing the "author" property in favor of "description", right? @shiffman

I think that makes sense, @klinegareth any reason you can think of why we might need a specific "author" field?

@augustkline
Copy link
Collaborator

@shiffman No, I think removing it is okay!

@fdoflorenzano
Copy link
Collaborator

Closing in favor of #69

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

No branches or pull requests

3 participants