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

ebayui-core-4.5.0 : ebayui-listbox-button should allow class update on re-render #1171

Closed
uachladi opened this issue Jul 13, 2020 · 9 comments

Comments

@uachladi
Copy link

uachladi commented Jul 13, 2020

Bug Report

eBayUI Version: 4.5.0

Description

ebay-listbox-button allows to pass class attribute but does not allow change of class on re-render due to w-preserve-attrs.
https://github.com/eBay/ebayui-core/blob/4.5.0/src/components/ebay-listbox-button/template.marko#L15

@dpiercey @ianmcburnie . This is a high priority for us to get the sellerhub skin upgrade to be completed.

@ianmcburnie
Copy link
Contributor

ianmcburnie commented Jul 13, 2020

@uachladi We'll look into it. Just curious to know what kind of class are you changing? Is it a visual change? Or some kind of JS hook? We like to learn how people are using the API :-)

@uachladi
Copy link
Author

@ianmcburnie , its just the text color.
Screen Shot 2020-07-13 at 3 59 41 PM

@ianmcburnie
Copy link
Contributor

@uachladi Thanks. What does the red colour convey? I'm concerned about accessibility and using colour alone to convey meaning. Actually, I'll ping you offline to chat further...

@ianmcburnie
Copy link
Contributor

@uachladi just checking in... do you still need this?

@uachladi
Copy link
Author

@ianmcburnie , yes product wants to keep this.

@ianmcburnie ianmcburnie added this to To do in 4.5.x Aug 18, 2020
@agliga
Copy link
Contributor

agliga commented Sep 1, 2020

@uachladi Is this needed on the 4.5.x branch or are you able to upgrade to 5.x` branch in the near future?

@uachladi
Copy link
Author

uachladi commented Sep 2, 2020

We do not have upgrade in our planning until Q2 2021. It will be needed on 4.5.x branch.

@agliga
Copy link
Contributor

agliga commented Sep 22, 2020

This is also needed in 5.x branch since it has no-update on class for listbox.

@agliga
Copy link
Contributor

agliga commented Sep 23, 2020

Released in 4.5.22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
4.5.x
  
Done
Development

No branches or pull requests

3 participants