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: Add caret icons #2

Closed
wants to merge 1 commit into from
Closed

feat: Add caret icons #2

wants to merge 1 commit into from

Conversation

johnletey
Copy link
Member

@johnletey johnletey requested a review from moeenio June 11, 2020 07:51
@moeenio
Copy link
Contributor

moeenio commented Jun 11, 2020

Not sure where this would be used

@johnletey
Copy link
Member Author

Not sure where this would be used

@locness3 Good point 🤔 Pretty much all of the use-cases that come to mind rn you can use the chevron icons 🤷‍♂️

@ericfennis
Copy link
Member

Nice I like them.
But @locness3 has a point, are those icons not the same use cases as the chevron icons ?

@johnletey
Copy link
Member Author

Fair, I agree with you both - I'll go ahead and close this 😄

@johnletey johnletey closed this Jun 17, 2020
@ericfennis ericfennis deleted the feat/caret branch September 22, 2020 12:39
ericfennis added a commit that referenced this pull request Oct 20, 2020
* Update (#2)

update

* Update README.md

* Fix links

* Fix typo

Co-authored-by: Eric Fennis <eric.fennis@gmail.com>

Co-authored-by: Locness <37651007+locness3@users.noreply.github.com>
Co-authored-by: Eric Fennis <eric.fennis@gmail.com>
ericfennis added a commit that referenced this pull request Dec 30, 2020
* Update (#2)

update

* Update README.md

* Fix links

* Fix typo

Co-authored-by: Eric Fennis <eric.fennis@gmail.com>

* Create brush.svg

* Add brush tags

* Edit brush

Co-authored-by: Locness <37651007+locness3@users.noreply.github.com>
Co-authored-by: Eric Fennis <eric.fennis@gmail.com>
@ericfennis ericfennis mentioned this pull request May 18, 2021
ericfennis added a commit that referenced this pull request Aug 10, 2022
* add pnpm

* make it work

* fix comamnds in package.jsons

* move some scripts to modules

* workflow fixes

* test workflow

* test #2

* minor fix

* update lockflite

* create workflows

* update workflow

* Add copy license command

* Fix build

* update workflows

* update contributions.md

* migrate site directory to pnpm

* Fix peer dependencies when install

* fix types in lucide-angular

* fix testing
@karsa-mistmere karsa-mistmere mentioned this pull request May 3, 2023
ericfennis added a commit that referenced this pull request Jul 1, 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