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

[Bug Report] Case insensitive dynamic slot #10002

Closed
dionysiosarvanitis opened this issue Dec 18, 2019 · 7 comments
Closed

[Bug Report] Case insensitive dynamic slot #10002

dionysiosarvanitis opened this issue Dec 18, 2019 · 7 comments

Comments

@dionysiosarvanitis
Copy link
Contributor

Environment

Vuetify Version: 2.1.15
Vue Version: 2.6.11
Browsers: Chrome 80.0.3987.7
OS: Windows 10

Steps to reproduce

Try to customize default row in a data-table where the header value contains an uppercase letter.

Expected Behavior

According to data-tables#customizing-default-rows the value should be customizable.

Actual Behavior

Value cannot be referred.

Reproduction Link

https://codepen.io/arvanoid/pen/NWPdgNQ?&editable=true&editors=101

Other comments

My use case is while trying to sort the header, I wanted to keep the value exactly as it is expected at the server side. For example, a url containing &sort=product.deliveryDate

@ghost ghost added the S: triage label Dec 18, 2019
@KaelWD
Copy link
Member

KaelWD commented Dec 18, 2019

@dionysiosarvanitis
Copy link
Contributor Author

Doesn't work with kebap-case equivelent @KaelWD

@jacekkarczmarczyk
Copy link
Member

Yes, it works https://codepen.io/jkarczm/pen/LYExzzb

If you have any additional questions, please reach out to us in our Discord community.

@MajesticPotatoe
Copy link
Member

It does: https://codepen.io/majesticpotatoe-the-bashful/pen/dyPNVZv
You cant use capitals in slot names (aka camelCase), as they are normalized to all-lowercase by the browser.

@dionysiosarvanitis
Copy link
Contributor Author

@MajesticPotatoe @jacekkarczmarczyk You've also changed the header value to be with dashes. I want the header value to be camelCase'd so I can use it as a sort field like that &sort=product.deliveryDate.

Like that https://codepen.io/arvanoid/pen/YzPNEvd

@MajesticPotatoe
Copy link
Member

@dionysiosarvanitis that was because you said kebab case doesn't work but it does...

You cant use capitals in slot names (aka camelCase), as they are normalized to all-lowercase by the browser

If you have any additional questions, please reach out to us in our Discord community.

@vuetifyjs vuetifyjs locked as resolved and limited conversation to collaborators Dec 18, 2019
@KaelWD
Copy link
Member

KaelWD commented Jan 8, 2020

To be clear:

You can't use capitals in slot names (aka camelCase) in DOM templates. Mixed case is fine in string templates (actual strings or <script type="text/x-template">) or .vue files.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants