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

Create lorem-standalone.js #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

hewiefreeman
Copy link

I've slightly modified the script for people like me who want to use this library as a way dynamically insert Lorem Ipsum into their JavaScript/web app's interface/mock.

I also added a new "TITLE" type to generate titles (for my own and other's needs) :)

This is meant to be contributed as a separate file for the purposes mentioned above. I'd also be happy to write up a read-me for this!

I've slightly modified the script for people like me who want to use this library as a way dynamically insert Lorem Ipsum into their JavaScript/web app's interface/mock.

I also added a new "TITLE" type to generate titles (for my own and other's needs) :)

This is meant to be contributed as a separate file for the purposes mentioned above. I'd also be happy to write up a read-me for this!
@olfek
Copy link
Contributor

olfek commented Aug 24, 2018

I've slightly modified the script for people like me who want to use this library as a way dynamically insert Lorem Ipsum into their JavaScript/web app's interface/mock.

The existing code already does this, what's the difference?

@f
Copy link
Owner

f commented Aug 25, 2018

I got the idea. Maybe we can make an option to make it dom-free and add a title mode instead of creating a whole new file to maintain.

@hewiefreeman
Copy link
Author

hewiefreeman commented Sep 9, 2018

@f Sounds like a better idea. I just made it like this for more readability and a smaller file for people who want to just use it to generate text in their own scripts. And the title option serves a nice purpose for most mock-ups.

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