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

library: Add Font Dialog Entry #244

Merged
merged 7 commits into from Jun 3, 2023
Merged

Conversation

SoNiC-HeRE
Copy link
Contributor

@SoNiC-HeRE SoNiC-HeRE commented Mar 22, 2023

Adds Font Dialog Entry
workbenchdev/demos#3

@sonnyp sonnyp self-assigned this Mar 22, 2023
@sonnyp sonnyp self-requested a review March 22, 2023 12:30
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.

The entry doesn't appear in the library

(re.sonny.Workbench.Devel:4): re.sonny.Workbench.Devel-WARNING **: 19:53:44.496: The demo name "Font Chooser" does not match the folder name "Font Dialog" and will be ignored.

console.log(`Font Dialog Button: The font selected is ${fontname}`);
});

const dialog_custom = new Gtk.FontChooserDialog({
Copy link
Contributor

Choose a reason for hiding this comment

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

this is the deprecated API

Copy link
Contributor Author

Choose a reason for hiding this comment

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

okay, I'll update this with the new one

@SoNiC-HeRE
Copy link
Contributor Author

I'll fix it

@sonnyp sonnyp force-pushed the main branch 3 times, most recently from 05f69f1 to 55e450f Compare May 21, 2023 10:37
@sonnyp sonnyp assigned SoNiC-HeRE and unassigned sonnyp May 28, 2023
@sonnyp
Copy link
Contributor

sonnyp commented May 28, 2023

@SoNiC-HeRE please make sure to update this to use FontDialog instead of the deprecated FontChooserDialog

@SoNiC-HeRE
Copy link
Contributor Author

SoNiC-HeRE commented Jun 1, 2023

I'm unsure why I'm getting an empty object or undefined value when i'm trying to get the value from Gtk.FontDialog. It seems to work fine with Gtk.FontDialogButton but not with a custom button. I'm using this for reference: https://docs.gtk.org/gtk4/class.FontDialog.html

@SoNiC-HeRE SoNiC-HeRE requested a review from sonnyp June 2, 2023 19:26
@SoNiC-HeRE
Copy link
Contributor Author

The demo seems to be working now , had to fix an error. It is ready for review now

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.

👍

made some minor changes 2321002

@sonnyp sonnyp merged commit 6ea4060 into workbenchdev:main Jun 3, 2023
sonnyp pushed a commit to krlade/Workbench that referenced this pull request Jun 3, 2023
sonnyp pushed a commit to SoNiC-HeRE/Workbench that referenced this pull request Aug 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants