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

font-tags: turn into local webapp #1013

Closed
wants to merge 1 commit into from
Closed

font-tags: turn into local webapp #1013

wants to merge 1 commit into from

Conversation

m4rc1e
Copy link
Collaborator

@m4rc1e m4rc1e commented Aug 5, 2024

The spreadsheets have become a nightmare to maintain so I've created a webapp that users run locally to edit font tags instead. The app works directly on the tagging csv so we can drop the spreadsheets.

To use the app run gftools font-tags path/to/google/fonts

@rsheeter
Copy link
Contributor

rsheeter commented Aug 5, 2024

I don't think we want users to run python, just give them a url. github pages (ala https://rsheeter.github.io/android_fonts/emoji.html?q=) or even codepen (poc https://codepen.io/rs42/pen/mdZwdZM) seems likely to be sufficient unless there is some capability that requires python?

console.error('Error:', error);
});
},
loadCSV() {
Copy link
Member

Choose a reason for hiding this comment

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

Can this be redone to use the Github API?

@davelab6
Copy link
Member

davelab6 commented Aug 5, 2024

unless there is some capability that requires python?

Well, the workflow Marc actually has is editing the file locally, so, in the httpd() call he has,

           "Once you're done editing, remember to commit the changes in "
           "your google/fonts repo and open a pr."

But all the py code is doing is loading the CSV and serving it via a httpd; this is useful for a local workflow since then the file is edited and can be directly committed.

For a github pages version, I think "all" Marc has to do is use the Github API (using something like https://github.com/elwayman02/ember-data-github) to load and commit the tags csv file directly. (Your plain js codepen example is good for loading the file, but presents a workflow challenge.)

Both workflows are worth supporting, imo

@rsheeter
Copy link
Contributor

rsheeter commented Aug 5, 2024

I would just let the user download either an updated file or a diff to use for a PR. That simplifies things and is a major improvement over the spreadsheet workflow. Most of the benefit at very low cost, near-0 maintenance because there are no dependencies or scary operations.

@m4rc1e
Copy link
Collaborator Author

m4rc1e commented Aug 5, 2024

SGTM. I'll drop the server and just host it on the google/fonts gh-pages.

@davelab6
Copy link
Member

davelab6 commented Aug 5, 2024

MVP :)

@rsheeter
Copy link
Contributor

rsheeter commented Aug 5, 2024

MVP :)

+1. Launch, begin to capture value, and iterate.

@m4rc1e
Copy link
Collaborator Author

m4rc1e commented Aug 8, 2024

Closing since this has moved to google/fonts, https://google.github.io/fonts/tags.html

@m4rc1e m4rc1e closed this Aug 8, 2024
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.

3 participants