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

feat(banners-lib): banner generation MVP #19

Draft
wants to merge 20 commits into
base: dev
Choose a base branch
from

Conversation

Guz013
Copy link
Member

@Guz013 Guz013 commented Nov 29, 2023

Create the Banners Library, adding at least the basic features of:

  • Title and Subtitle
  • Icon fetching from Iconify
  • Solid color background

Limitations that should be worked on:

  • Convert text in SVG to paths
  • Check types of the Banner Object
  • Read local SVG files without node:fs
  • Change title position when there's no title

Closes #13

@Guz013 Guz013 linked an issue Nov 29, 2023 that may be closed by this pull request
Copy link

stackblitz bot commented Nov 29, 2023

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copy link

changeset-bot bot commented Nov 29, 2023

⚠️ No Changeset found

Latest commit: 063e316

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@Guz013 Guz013 self-assigned this Nov 29, 2023
@Guz013 Guz013 added the enhancement New feature or request label Nov 29, 2023
@Guz013 Guz013 added this to the Banners library milestone Nov 29, 2023
Added to the SVG converting script a line to create .svg.js files.
These files are normal JS files that exports just a single string,
the SVG file content. This way the library doesn't need a build
step to embed these files and/or node:fs to read them.
Copy link

github-actions bot commented Dec 1, 2023

This pull request has been deployed to Vercel.

Latest commit: fc3243d
✅ Preview: https://lored-markdown-czwy2yel0-guz013.vercel.app
🔍 Inspect: https://vercel.com/guz013/lored-markdown/72biaWLKS9BrzYEt6UpQJKaGti6h

View Workflow Logs

Removed the text to path conversion, it was
too complicated to align the generated paths
with the dynamic layouts' texts. So embeded
font files using data URIs is being used,
this is experimental and a "quick-fix", because
data URIs could be not compatible and/or work
in some environments and browsers. A comment and/or
issue with more details about this will be created.
Uploaded the generated svg to be tested on
github and other environments
@Guz013
Copy link
Member Author

Guz013 commented Jan 17, 2024

test

time to reinvent the wheel again
@Guz013 Guz013 changed the title feat(banners-lib): banners library and basic banner generation feat(banners-lib): banner generation Jan 17, 2024
@Guz013 Guz013 changed the title feat(banners-lib): banner generation feat(banners-lib): banner generation MVP Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployed enhancement New feature or request
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

feat(banner-lib): Banner Object API
1 participant