Skip to content

Vala rewrite #6

Draft
Suzie97 wants to merge 29 commits intopongloongyeat:masterfrom
Suzie97:vala-rewrite
Draft

Vala rewrite #6
Suzie97 wants to merge 29 commits intopongloongyeat:masterfrom
Suzie97:vala-rewrite

Conversation

@Suzie97
Copy link

@Suzie97 Suzie97 commented Jul 5, 2021

This PR does the UI part of the vala rewrite of the app, and also installs the icon, and appdata.

I'm sorry I'm new to PRs and other things, so I don't know a lot. I don't have access create a branch in the repo so I'm directly opening a draft PR on master.

};
headerbar.get_style_context ().add_class (Gtk.STYLE_CLASS_FLAT);
headerbar.get_style_context ().add_class ("default-decoration");

Copy link
Author

Choose a reason for hiding this comment

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

The Granite.Widgets.Welcome widget was not used here because it uses a slightly darker background color which is not the same as the flat headerbar color. So it doesn't look like as if the headerbar is part of the window itself.

Copy link
Owner

@pongloongyeat pongloongyeat left a comment

Choose a reason for hiding this comment

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

Saw code, haven't actually compiled. I've made a vala-rewrite branch so when your PR is out of draft, you can request for this PR to be merged in there in the meantime.

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.

2 participants