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

Changed webidl generation to use data-lt attribute #469

Merged
merged 2 commits into from Aug 21, 2015
Merged

Conversation

halindrome
Copy link
Contributor

This should address issue #464, but I don't have any specs with
webidl in them to verify that! Can someone please take a close look?

This should address issue #464, but I don't have any specs with
webidl in them to verify that!
@marcoscaceres
Copy link
Member

@jyasskin can you please take a look?

@@ -720,7 +720,7 @@ define(
// Found it: update the definition to specify its [for] and title.
Copy link
Member

Choose a reason for hiding this comment

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

Update the comments too.

@jyasskin
Copy link
Member

jyasskin commented Aug 9, 2015

There's also a use of @data-title in run(). Can you double-check whether that needs to change?

Other than that and the comments, this looks good to me. Thanks.

@inexorabletash
Copy link
Member

@marcoscaceres - any chance for progress here?

@marcoscaceres
Copy link
Member

Merging this, if there are issues with @data-title, can fix them in a new bug. We should find out pretty quickly.

marcoscaceres pushed a commit that referenced this pull request Aug 21, 2015
Changed webidl generation to use data-lt attribute
@marcoscaceres marcoscaceres merged commit f46cf16 into develop Aug 21, 2015
@marcoscaceres marcoscaceres deleted the p464 branch August 21, 2015 20:36
@halindrome
Copy link
Contributor Author

Fwiw I looked at this and data-title is being used for something else.
Shouldn't be a problem.
On Aug 21, 2015 3:36 PM, "Marcos Caceres" notifications@github.com wrote:

Merging this, if there are issues with @data-title, can fix them in a new
bug. We should find out pretty quickly.


Reply to this email directly or view it on GitHub
#469 (comment).

@marcoscaceres
Copy link
Member

Thanks for checking that @halindrome

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

Successfully merging this pull request may close these issues.

None yet

4 participants