Skip to content
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.

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

Port 'CSS Gradients' demo to Vala #155

Merged
merged 4 commits into from
Sep 26, 2024

Conversation

BharatAtbrat
Copy link
Contributor

Intitial port for CSS gradients

  • Gtk Style Context was deprecated (since 4.10) and needs an alternative
  • Javascript and Python implementations also use the same

Intitial implementation
Gtk Style Context was deprecated (since 4.10) needs alternative
Javascript and Python also use the same
Copy link
Contributor

@Diego-Ivan Diego-Ivan left a comment

Choose a reason for hiding this comment

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

Looks great, thanks!

src/CSS Gradients/main.vala Outdated Show resolved Hide resolved
Remove setter function for language and
style_scheme
@BharatAtbrat BharatAtbrat requested a review from Diego-Ivan May 29, 2024 19:40
Copy link
Contributor

@lw64 lw64 left a comment

Choose a reason for hiding this comment

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

thanks, and better test my changes before pushing ;)

src/CSS Gradients/main.vala Outdated Show resolved Hide resolved
src/CSS Gradients/main.vala Outdated Show resolved Hide resolved
src/CSS Gradients/main.vala Outdated Show resolved Hide resolved
src/CSS Gradients/main.vala Outdated Show resolved Hide resolved
src/CSS Gradients/main.vala Outdated Show resolved Hide resolved
src/CSS Gradients/main.vala Outdated Show resolved Hide resolved
Fix comment Indentation
Replace if else with switch
Change css string syntax
Explicit declaration for css_language, scheme, clipboard
Changes recommended by @lw64
@BharatAtbrat BharatAtbrat requested a review from lw64 May 30, 2024 08:08
@sonnyp
Copy link
Contributor

sonnyp commented Jun 14, 2024

Needed to make a tweak to workbench cli workbenchdev/Workbench@d8f06f1

I will merge this before the next release so CI can keep passing.

@sonnyp
Copy link
Contributor

sonnyp commented Jun 14, 2024

@Diego-Ivan @lw64 can one of you review ?

Copy link
Contributor

@Diego-Ivan Diego-Ivan left a comment

Choose a reason for hiding this comment

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

Thanks!

@sonnyp sonnyp merged commit 2a6b18d into workbenchdev:main Sep 26, 2024
1 check failed
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.

4 participants