Skip to content

Conversation

@glebtv
Copy link

@glebtv glebtv commented Apr 7, 2017

This is needed for serializeable_meta because it wraps multiple fields with qor-field, so all fields will have the first one's template.

…-template

This is needed for serializeable_meta because it wraps multiple fields
with qor-field
@jinzhu jinzhu requested a review from jasonweng April 18, 2017 15:58
Copy link
Member

@jasonweng jasonweng left a comment

Choose a reason for hiding this comment

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

You don't have to add .first() selector, $this.parents('.qor-field') will find the closest .qor-field tag.

Please take a screenshot for your HTML structrue. i don't think .first() selector is necessary.

@glebtv
Copy link
Author

glebtv commented Apr 19, 2017

This in in "worker/jobs" page, arguments to a job.

.parents() finds not the first but all matching parents up to <body>, see https://jsfiddle.net/8e4u1zou/

@jasonweng
Copy link
Member

Use .closest('. qor-field') is works for you?

@theplant-ci
Copy link

@glebtv: PR needs rebase.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants