Skip to content

Conversation

@krlade
Copy link
Contributor

@krlade krlade commented Mar 7, 2023

This pull request is still work in progress. I'm opening this to indicate that some work is being done and to avoid duplicating efforts.

The goal of this pr is to create a demo for a boxed list, mainly focuses on AdwPreferencesGroup, AdwActionRow, AdwComboRow and AdwEntryRow with an example use cases.

Related to workbenchdev/demos#3

@sonnyp sonnyp changed the title Draft: Library: Add Row entry library: Add Row entry Mar 7, 2023
@sonnyp sonnyp self-requested a review March 11, 2023 22:59
@sonnyp sonnyp self-assigned this Mar 11, 2023
@sonnyp sonnyp removed their request for review March 12, 2023 23:23
@sonnyp
Copy link
Contributor

sonnyp commented Mar 16, 2023

How is it going? Don't hesitate to let us know in the chatroom if you have any question.

@krlade
Copy link
Contributor Author

krlade commented Mar 17, 2023

How is it going? Don't hesitate to let us know in the chatroom if you have any question.

Sorry that it takes so long I almost forgot about this pr. I'll try to complete it soon. I've got few minor issues, I'll ask it on the chat later

@krlade krlade force-pushed the row-demo branch 2 times, most recently from 449a5f4 to 3019a2d Compare March 17, 2023 21:35
@krlade krlade marked this pull request as ready for review March 17, 2023 21:36
@krlade
Copy link
Contributor Author

krlade commented Mar 17, 2023

It should be ready for initial review

Copy link
Contributor

@sonnyp sonnyp left a comment

Choose a reason for hiding this comment

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

Good start.

Please add 18px margin between the sections

@krlade krlade changed the title library: Add Row entry library: Add Boxed lists entry Mar 21, 2023
@krlade krlade requested a review from sonnyp March 24, 2023 11:51
Copy link
Contributor

@sonnyp sonnyp left a comment

Choose a reason for hiding this comment

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

There are quite a lot more type of rows but it would be too big in this one entry.

Let's get this in and see how we can extend/improvate later 👍

@sonnyp sonnyp force-pushed the main branch 3 times, most recently from 05f69f1 to 55e450f Compare May 21, 2023 10:37
@sonnyp
Copy link
Contributor

sonnyp commented May 28, 2023

hey @karl0d, do you want to finish this? If so, make sure to add your name to the list of contributors in about.js

@sonnyp sonnyp assigned krlade and unassigned sonnyp May 28, 2023
@krlade
Copy link
Contributor Author

krlade commented May 28, 2023

Yes I want to finish it, it's almost ready. I will push my changes soon

@krlade
Copy link
Contributor Author

krlade commented May 28, 2023

Should this demo cover new widgets introduced in libadwaita 1.4 such as Adw.SpinRow and Adw.SwitchRow? If not so, there is only one issue blocking this pr from being ready

@krlade krlade requested a review from sonnyp May 28, 2023 16:14
@sonnyp
Copy link
Contributor

sonnyp commented Jun 3, 2023

Sorry for the late answer, I missed the notification somehow.

Should this demo cover new widgets introduced in libadwaita 1.4 such as Adw.SpinRow and Adw.SwitchRow?

No, it's not available in Workbench.

When GNOME 45 releases (with libadwaita 1.4) we will make a list of new things we want to demo in Workbench.

Copy link
Contributor

@sonnyp sonnyp left a comment

Choose a reason for hiding this comment

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

Thanks!

Please avoid force pushing, it makes it hard to review, specially after some time coming back to this.

I made a few changns https://github.com/sonnyp/Workbench/pull/210/files

  • The ListBox should be the parent of rows
  • selection-mode: none; and styles ["boxed-list"] was missing on ListBox as documented by libadwaita
  • Since we now have a "Preferences Window" demo, I removed all Preferences related widgets

@sonnyp sonnyp merged commit a9b2ab8 into workbenchdev:main Jun 3, 2023
sonnyp pushed a commit that referenced this pull request Jun 10, 2023
sonnyp pushed a commit to SoNiC-HeRE/Workbench that referenced this pull request Aug 13, 2023
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.

5 participants