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

Add Prepend method to Accordion #5418

Closed
2 tasks done
JeffreyLeeLi opened this issue Jan 16, 2025 · 4 comments
Closed
2 tasks done

Add Prepend method to Accordion #5418

JeffreyLeeLi opened this issue Jan 16, 2025 · 4 comments

Comments

@JeffreyLeeLi
Copy link

Checklist

  • I have searched the issue tracker for open issues that relate to the same feature, before opening a new one.
  • This issue only relates to a single feature. I will open new issues for any other features.

Is your feature request related to a problem?

I'm frustrated to manipulate Items field directly, 'coz it'll panic.

Is it possible to construct a solution with the existing API?

None works for me now.

Describe the solution you'd like to see.

func (a *Accordion) Prepend(item *AccordionItem)

@Jacalz
Copy link
Member

Jacalz commented Jan 16, 2025

I'm not necessarily saying that I'm against adding this, but I want to clarify that "coz it'll panic" is not a good enough reason for adding this. As explained in the previous issue, it panics because of a race condition that that has very likely been fixed on the develop branch for 2.6.

@Jacalz Jacalz changed the title add Prepend method to Accordion Add Prepend method to Accordion Jan 16, 2025
@andydotxyz
Copy link
Member

This sounds like a reasonable addition - other widgets with Append have added Prepend too

@andydotxyz andydotxyz added this to the "F" release, Early 2025 milestone Jan 16, 2025
@JeffreyLeeLi
Copy link
Author

Thanks for the adoption.

@andydotxyz
Copy link
Member

Added on develop branch ready for 2.6.0 release

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

No branches or pull requests

3 participants