Skip to content
This repository was archived by the owner on Jun 4, 2024. It is now read-only.

add basic HTML serializer for header and footer #29

Closed
wants to merge 3 commits into from
Closed

Conversation

chriddyp
Copy link
Member

No description provided.

chriddyp added a commit to plotly/dash that referenced this pull request Nov 30, 2017
this depends on html components having a HTML serializer, see
plotly/dash-html-components#29
@chriddyp
Copy link
Member Author

This PR enables server-rendered HTML tags in a dash app's app.head and app.footer, new attributes introduced in plotly/dash#171

# convert e.g. fontColor to font-color
inline_style = '; '.join([
'='.join([_camel_case_to_css_case(k), v])
for (k, v) in value.iteritems()

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems as though the = should be an :

@chriddyp
Copy link
Member Author

chriddyp commented Jun 8, 2018

An interesting idea, but not quite the path I want to head down anymore. The purpose of this PR was to solve plotly/dash#171 which is now replaced with the proposal in plotly/dash#265

@chriddyp chriddyp closed this Jun 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants