Skip to content

Conversation

@BharatAtbrat
Copy link
Contributor

@BharatAtbrat BharatAtbrat commented Jul 4, 2024

Initial implementation to remove usage of Adw.PreferencesPage and Adw.PreferencesGroup

  • Design is still WIP

Current bugs :

  • Labels for categories are added as a ListBox Row, design needs changes
  • Increasing window size increases the size of rows (ideal -> row length remains constant and only reduces in length when window is minimized)
  • New issue after latest commit when searching for a demo, the size of the rows is set by looking at the row with maximum length
  • Hide categories when searching [WIP]

Screengrab after commit 3

2024-07-05.19-36-10.mp4

After commit 4

2024-07-05.23-06-11.mp4

Initial implementation
-Design is still WIP
Current bugs :
-Labels for categories are added as a ListBox Row, design needs changes
-Increasing window size increases the size of rows (ideal -> row length
remains constant and only reduces in length when window is minimized)
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.

Please replica the previous design for now

Should be something like this

Box {
  Label: {
    label: "User Interface";
    styles ["title-4"]
  }
  
  ListBox library_user_interface {
    ...
  }

}
```

@sonnyp
Copy link
Contributor

sonnyp commented Jul 4, 2024

Also, categories should be hidden on search

BharatAtbrat and others added 4 commits July 5, 2024 19:31
Hide categories on search
Remove separate boxes for each listbox + label combination
Margins use hardcoded values to look similar to current Workbench
@sonnyp sonnyp marked this pull request as ready for review July 6, 2024 23:48
@sonnyp sonnyp merged commit f4eb0e6 into main Jul 6, 2024
@sonnyp sonnyp deleted the atbrat/pagegroupfixes branch July 6, 2024 23:48
@sonnyp sonnyp mentioned this pull request Jul 6, 2024
10 tasks
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

Successfully merging this pull request may close these issues.

3 participants