#2896

Add support for `defaultValue` option for `puter.ui.showFontPicker` in puter app environment

Openreynaldichernandoopened 5mo ago

Currently the defaultValue options only works in web environment. Ideally we match this functionality in puter apps env

app

web

Also somehow our font picker UI is different in web and in puter app env

Best to try and compare it here https://ui-playground.puter.site/ https://puter.com/app/ui-playground

Activity

10
10jake105mo ago

I would like to work on this issue

jelveh5mo ago

Go for it! assigned

SusTechStitch5mo ago

Hi, I noticed there is no PR for this yet. I would like to work on supporting defaultValue for puter.ui.showFontPicker in the Puter app environment. Is it okay if I submit a PR for this?

10jake105mo ago

Fine by me. I was still going to get to it eventually but got distracted by other things. Go for it.

yogeshsince20232mo ago

Is this issue still open to work ? @HeyPuter

Salazareo2mo ago

hi @yogeshsince2023 sorry for the delay, yes, if you are able to grab it I'm happy to assign it to you

M001N2mo ago

I'd like to work on this - root cause is that UIWindowFontPicker.js (app/embedded path) only reads options.default, never defaultFont/defaultValue, while UI.js's standalone fallback ignored defaultValue too; I have a small fix ready in both files and will open a PR shortly.

Salazareo2mo ago

oh cool @M001N ill hand it to you then

Bhava-hub1mo ago

hey is this issue still open

anishcodes-0-918d ago

Hi @Salazareo , I'd like to contribute to Puter and was looking into #2896. I noticed M001N mentioned having a fix ready in August, but I couldn't find a linked PR. Is this still being worked on, or would it be possible for me to take it forward?