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

Add a note about finishing touches to "Set CSS Name and Use Exported Colors" section of 14th (CSS) chapter of the book. #1339

Conversation

zekefast
Copy link
Contributor

Changes

  • Remind reader to include style.css to resources.gresource.xml and call load_css() function in connect_startup handler.
  • Add anchors to listings/todo/3/main.rs file to be able to show them in the chapter:
    • connect_startup
    • load_css

Issues

…Colors" section of 14th (CSS) chapter of the book.

Changes:
- Remind reader to include "style.css" to "resources.gresource.xml" and
  call "load_css()" function in `connect_startup` handler.
- Add anchors to listings/todo/3/main.rs file to be able to show them
  in the chapter:
  - connect_startup
  - load_css
@zekefast
Copy link
Contributor Author

@Hofer-Julian Could you please visually verify that my changes okay before merging PR? I haven't rendered the book with them. 😊

@Hofer-Julian
Copy link
Collaborator

@Hofer-Julian Could you please visually verify that my changes okay before merging PR? I haven't rendered the book with them. 😊

Please render it locally before opening pull requests. Do you have trouble with setting up the development environment?

book/src/css.md Outdated Show resolved Hide resolved
book/src/css.md Outdated Show resolved Hide resolved
book/src/css.md Outdated Show resolved Hide resolved
@zekefast
Copy link
Contributor Author

@Hofer-Julian

Please render it locally before opening pull requests.

I will do. Sorry for that.

Do you have trouble with setting up the development environment?

It seems all working fine and https://github.com/gtk-rs/gtk4-rs/tree/master/book#build-instructions is quite sufficient. It just was a night and a bit of laziness on my side. :)

Rendered addition looks fine to me.

Screenshot from 2023-03-20 11-28-27

Copy link
Collaborator

@Hofer-Julian Hofer-Julian left a comment

Choose a reason for hiding this comment

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

Looks good otherwise :)

book/src/css.md Outdated Show resolved Hide resolved
book/src/css.md Outdated Show resolved Hide resolved
book/src/css.md Outdated Show resolved Hide resolved
book/src/css.md Show resolved Hide resolved
@bilelmoussaoui
Copy link
Member

@Hofer-Julian

Please render it locally before opening pull requests.

I will do. Sorry for that.

Do you have trouble with setting up the development environment?

It seems all working fine and https://github.com/gtk-rs/gtk4-rs/tree/master/book#build-instructions is quite sufficient. It just was a night and a bit of laziness on my side. :)

Rendered addition looks fine to me.

Screenshot from 2023-03-20 11-28-27

I see there is a missing space on the diff (green line), do you mind adding it?

@zekefast
Copy link
Contributor Author

zekefast commented Mar 20, 2023

I see there is a missing space on the diff (green line), do you mind adding it?

@Hofer-Julian It was removed by @bilelmoussaoui, but I'll add it back.

Current look

Screenshot 2023-03-20 at 14-18-51 CSS - GUI development with Rust and GTK 4

zekefast and others added 2 commits March 20, 2023 14:11
Co-authored-by: Bilal Elmoussaoui <belmouss@redhat.com>
Co-authored-by: Bilal Elmoussaoui <belmouss@redhat.com>
@zekefast zekefast force-pushed the book_add_reminder_about_loading_css_to_14th_chapter branch from bb8e75c to d71f36b Compare March 20, 2023 13:12
Copy link
Collaborator

@Hofer-Julian Hofer-Julian left a comment

Choose a reason for hiding this comment

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

Thank you!

@Hofer-Julian Hofer-Julian merged commit 75d248c into gtk-rs:master Mar 20, 2023
3 checks passed
@zekefast
Copy link
Contributor Author

@Hofer-Julian Thank you for merging it!

@zekefast zekefast deleted the book_add_reminder_about_loading_css_to_14th_chapter branch March 20, 2023 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants