Skip to content

version 1.0.0 #53

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.

Sign up for GitHub

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

Jump to bottom
Merged
prateekmedia merged 27 commits into from
Feb 7, 2022
Merged

version 1.0.0 #53

prateekmedia merged 27 commits into from
Feb 7, 2022

Conversation

@prateekmedia
Copy link
Member

@prateekmedia prateekmedia commented Jan 20, 2022 •
edited
Loading

BREAKING

  • AdwHeaderBar.minimal is now AdwHeaderBar.custom
  • Remove label parameter from AdwTextField

Changes to widgets

HeaderBar

  • Now the AdwHeaderBar is not dependent on any package, windowDecor object is now optional
  • Add isTransparent parameter => Makes AdwHeaderBar's background and border color

Popover

ViewSwitcher

  • Add badge in AdwViewSwitcher

New Widgets

  • AdwSwitch => port of GtkSwitch from gtk4
  • AdwAboutWindow => port of upcoming AdwAboutWindow from libadwaita

@prateekmedia prateekmedia requested review from MalcolmMielle, jesusrp98 and simrat39 and removed request for jesusrp98 and simrat39 January 23, 2022 19:00
@MalcolmMielle
Copy link
Collaborator

MalcolmMielle commented Jan 23, 2022

I'll have a look as soon as I can :)

simrat39
simrat39 requested changes Jan 24, 2022
View reviewed changes
@prateekmedia
Break some stuff by renaming minimal to custom in HeaderBar
9b5fe05 
Don't require window_decorations package
Use Custom Painter for rendering window buttons, add minimize btn
@prateekmedia prateekmedia changed the title Default libadwaita's header buttons feat(headerbar): update header buttons Jan 24, 2022
@prateekmedia prateekmedia marked this pull request as ready for review January 27, 2022 11:21
@prateekmedia prateekmedia changed the title feat(headerbar): update header buttons A Big Update Jan 29, 2022
@prateekmedia prateekmedia requested a review from simrat39 January 29, 2022 05:17
@prateekmedia prateekmedia changed the title A Big Update version 1.0.0 Jan 29, 2022
MalcolmMielle
MalcolmMielle requested changes Jan 29, 2022
View reviewed changes
Copy link
Collaborator

@MalcolmMielle MalcolmMielle left a comment

Choose a reason for hiding this comment

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

Sorry I don't have more time to go through the code right now :S. I didn't have time to run the code itself.

Great work! A lot of good stuff in this commit and a lot of very needed addition to the package :D!
I've added some comments, mainly cosmetic, comments, and some about the logic.

Thanks a again for the great PR!

pablojimpas
pablojimpas reviewed Feb 4, 2022
View reviewed changes
pablojimpas
pablojimpas reviewed Feb 4, 2022
View reviewed changes
@prateekmedia
Copy link
Member Author

prateekmedia commented Feb 6, 2022

I think it is ready to merge, I will ask for a final review if any and then it will be merged this week or so.

@pablojimpas
Copy link
Contributor

pablojimpas commented Feb 7, 2022

I think it is ready to merge, I will ask for a final review if any and then it will be merged this week or so.

I suggest we should hold the 1.0.0 release until we find a right solution to issue #46

I haven't looked for ways to fix it but I think it's worth fixing this before a "stable" release, because a) it's bad API design and b) users of this package may expect some stability and not a lot of breaking changes in AdwSidebar which is probably one of the most essential & common widgets. There's no need to rush a release.

@MalcolmMielle
Copy link
Collaborator

MalcolmMielle commented Feb 7, 2022

While I agree with @pablojimpas that fixing #46 should come before stable release if possible, I think this PR should be merged when done and the fix to #46 should be in a different PR. It'll make it easier to review.

@prateekmedia
Copy link
Member Author

prateekmedia commented Feb 7, 2022

I am soon going to create another PR to fix #46 before stable release.

@prateekmedia prateekmedia merged commit 66dc9fb into main Feb 7, 2022
@prateekmedia
Copy link
Member Author

prateekmedia commented Feb 7, 2022 •
edited
Loading

PR for fixing #46 is #56

@prateekmedia prateekmedia deleted the add_window_buttons branch February 7, 2022 15:55
@pablojimpas
Copy link
Contributor

pablojimpas commented Feb 7, 2022

PR for fixing #46 is #56

Good to hear that! Maybe you can use some insight from @jesusrp98 since I think he did some research on this topic

@prateekmedia
Copy link
Member Author

prateekmedia commented Feb 7, 2022

@pablojimpas Hmm, he didn't posted anything after that so maybe he got busy, Idk.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jesusrp98 jesusrp98 Awaiting requested review from jesusrp98

@simrat39 simrat39 Awaiting requested review from simrat39

@MalcolmMielle MalcolmMielle Awaiting requested review from MalcolmMielle

1 more reviewer

@pablojimpas pablojimpas pablojimpas left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@prateekmedia @MalcolmMielle @pablojimpas @simrat39