-
Notifications
You must be signed in to change notification settings - Fork 6
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
Embed question page #88
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
(from #77)
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.
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.
None. Let's wait for user requests instead of trying to guess. Discoverability: I added an Overall, question page layout is becoming messy, I'm going to experiment a bit with different headings and alignments before merging. |
This is beautiful |
@Claygr, check it out |
(See also: ForumMagnum/ForumMagnum#4907 and ForumMagnum/ForumMagnum#4728 for how to integrate this with LW) |
These links are identical, should one of them be different? |
Yeah, changed |
Some updates:
(I plan to implement LW embeds and #89 in separate PRs, this one is for embed feature only) |
Also: section anchor links, because why not. |
Oops, my push failed due to rebase and I haven't noticed. Will open a separate PR. |
For #77.