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

feat: trees and lists #264

Open
wants to merge 80 commits into
base: master
Choose a base branch
from
Open

feat: trees and lists #264

wants to merge 80 commits into from

Conversation

caarlos0
Copy link
Member

@caarlos0 caarlos0 commented Feb 16, 2024

Worked on top of @maaslalani trees and lists implementations, fixed some bugs, refactor some stuff, added a couple of possibilities (namely, sublists (as trees)), etc.

Its a big PR, but it if it makes it better, it closes this 4 other PRs:

closes #242
closes #240
closes #243
closes #263

Some examples:

CleanShot 2024-02-16 at 10 30 22@2x
CleanShot 2024-02-16 at 10 31 00@2x
CleanShot 2024-02-16 at 10 31 25@2x
CleanShot 2024-02-16 at 10 32 43@2x
CleanShot 2024-02-16 at 11 08 51@2x

maaslalani and others added 28 commits November 30, 2023 10:50
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
@caarlos0 caarlos0 added the enhancement New feature or request label Feb 16, 2024
@caarlos0 caarlos0 self-assigned this Feb 23, 2024
@bashbunni
Copy link
Member

This is really cool! Ran all the examples on my machine and it looks rad 🥳

@caarlos0 caarlos0 requested a review from muesli as a code owner March 8, 2024 19:30
@caarlos0 caarlos0 requested a review from maaslalani March 8, 2024 19:31
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
@testinfected
Copy link

Hey Charm team, this is awesome. When can we expect it to land into master? That's exactly what we're missing.

Thanks!

tree/item.go Outdated Show resolved Hide resolved
tree/item.go Outdated Show resolved Hide resolved
tree/item.go Outdated Show resolved Hide resolved
tree/item.go Outdated Show resolved Hide resolved
Copy link
Member

@aymanbagabas aymanbagabas left a comment

Choose a reason for hiding this comment

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

Lgtm

list/enumerations.go Outdated Show resolved Hide resolved
caarlos0 and others added 5 commits April 18, 2024 08:16
Co-authored-by: Ayman Bagabas <ayman.bagabas@gmail.com>
Co-authored-by: bashbunni <15822994+bashbunni@users.noreply.github.com>
* chore(examples): go mod tidy

* feat(tree): add rounded corner enumerator
@bashbunni
Copy link
Member

Hey @testinfected we're still doing some usability tests on the API to decide if the developer experience is where we want it to be before release. No ETA yet, but the functionality is there, just want to make sure it's easy to use before publishing. We don't want to have to break the API in the future, so now's the time to get it all sorted :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants