search

quantified-uncertainty / metaforecast

Fetch forecasts from prediction markets/forecasting platforms to make them searchable. Integrate these forecasts into other services.
https://metaforecast.org/
MIT License
56 stars 5 forks source link

Embed question page #88

Closed berekuk closed 2 years ago

berekuk commented 2 years ago

For #77.

vercel[bot] commented 2 years ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
metaforecast ✅ Ready (Inspect) Visit Preview May 26, 2022 at 1:15PM (UTC)
berekuk commented 2 years ago

(from #77)

Things to consider:

  • Size

Responsive, with recommended 600x600 pixels in html code, which should be enough to fit the chart and options, probably with a scrollbar, but options should fit in a visible area.

  • Caching?

None; I'm not sure if our database will handle the load if someone embeds the chart on >10 rps page, but I think it's ok not to optimize this for now.

  • Customizability?

None. Let's wait for user requests instead of trying to guess.

Discoverability: I added an Embed section on each question's page.

Overall, question page layout is becoming messy, I'm going to experiment a bit with different headings and alignments before merging.

NunoSempere commented 2 years ago

This is beautiful

NunoSempere commented 2 years ago

@ClayGr, check it out

NunoSempere commented 2 years ago

(See also: https://github.com/ForumMagnum/ForumMagnum/pull/4907 and https://github.com/ForumMagnum/ForumMagnum/pull/4728 for how to integrate this with LW)

berekuk commented 2 years ago

https://github.com/ForumMagnum/ForumMagnum/pull/4907 and https://github.com/ForumMagnum/ForumMagnum/pull/4907

These links are identical, should one of them be different?

NunoSempere commented 2 years ago

Yeah, changed

berekuk commented 2 years ago

Some updates:

(I plan to implement LW embeds and #89 in separate PRs, this one is for embed feature only)

berekuk commented 2 years ago

Also: section anchor links, because why not.

berekuk commented 2 years ago

Oops, my push failed due to rebase and I haven't noticed. Will open a separate PR.