Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Aug 29, 2025. It is now read-only.

June 2019 update component gen, and save updated attributes.html#119

Merged
alexcjohnson merged 1 commit intomasterplotly/dash-html-components:masterfrom
update-jun-2019-bplotly/dash-html-components:update-jun-2019-bCopy head branch name to clipboard
Jun 18, 2019
Merged

June 2019 update component gen, and save updated attributes.html#119
alexcjohnson merged 1 commit intomasterplotly/dash-html-components:masterfrom
update-jun-2019-bplotly/dash-html-components:update-jun-2019-bCopy head branch name to clipboard

Conversation

@alexcjohnson
Copy link
Collaborator

supersedes #118 - without build artifacts, thx @Marc-Andre-Rivet

Noticed a couple of additions when I reran generate-components just now: for Button, formEncType, formMethod, formTarget; for Select: autoComplete. They seem reasonable...

While I was at it I had the script overwrite data/attributes.html so we can see the diff there (though it's pretty messy)

* If the button/input is a submit button (type="submit"), this attribute specifies the browsing context (for example, tab, window, or inline frame) in which to display the response that is received after submitting the form. If this attribute is specified, it overrides the target attribute of the button's form owner.
*/
'formTarget': PropTypes.string,

Copy link
Contributor

Choose a reason for hiding this comment

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

I agree with #118 (comment), we could indeed rename or exclude these files from the commits at some point. The only argument for keeping that at this point is that we do not fully trust our generation & testing steps yet.

Copy link
Contributor

@Marc-Andre-Rivet Marc-Andre-Rivet left a comment

Choose a reason for hiding this comment

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

💃 Looks fine to me.

@alexcjohnson alexcjohnson merged commit 09c3ddf into master Jun 18, 2019
@alexcjohnson alexcjohnson deleted the update-jun-2019-b branch June 18, 2019 19:34
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.