Skip to content

Conversation

@halfmexican
Copy link
Contributor

we aren't deprecated yet!

Copy link
Contributor

@andyholmes andyholmes left a comment

Choose a reason for hiding this comment

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

Nice one! Just a few tweaks and it LGTM.

halfmexican and others added 3 commits June 15, 2023 14:19
Fix formatting and remove stuff

Co-authored-by: Andy Holmes <1265208+andyholmes@users.noreply.github.com>
@halfmexican
Copy link
Contributor Author

a few tweaks have been tweaked

@halfmexican halfmexican requested a review from andyholmes June 15, 2023 19:24
@sonnyp sonnyp removed their request for review June 15, 2023 21:14
Copy link
Contributor

@andyholmes andyholmes left a comment

Choose a reason for hiding this comment

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

LGTM

@andyholmes andyholmes merged commit 7689965 into main Jun 15, 2023
@halfmexican halfmexican deleted the halfmexican/carousel-example branch June 17, 2023 17:20
sonnyp pushed a commit that referenced this pull request Jun 20, 2023
* library: add carousel entry
* make carousel functional
* add main.vala
* fix horizontal orientation
* fix alignment on prefsrow
* main.vala: fix formatting
* Apply suggestions from code review

   Fix formatting and remove stuff
   Co-authored-by: Andy Holmes <1265208+andyholmes@users.noreply.github.com>
* main.blp: fix spacing
* remove quotes

---------

Co-authored-by: wlhunter <wlhunter@go.olemiss.edu>
Co-authored-by: Andy Holmes <1265208+andyholmes@users.noreply.github.com>
sonnyp pushed a commit to SoNiC-HeRE/Workbench that referenced this pull request Aug 13, 2023
* library: add carousel entry
* make carousel functional
* add main.vala
* fix horizontal orientation
* fix alignment on prefsrow
* main.vala: fix formatting
* Apply suggestions from code review

   Fix formatting and remove stuff
   Co-authored-by: Andy Holmes <1265208+andyholmes@users.noreply.github.com>
* main.blp: fix spacing
* remove quotes

---------

Co-authored-by: wlhunter <wlhunter@go.olemiss.edu>
Co-authored-by: Andy Holmes <1265208+andyholmes@users.noreply.github.com>
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