Skip to content
This repository has been archived by the owner on Sep 10, 2022. It is now read-only.

Update ReadMe File With Adding A Scaling ScreenShot. #22

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

AbhiPrajapati
Copy link

I Have Created a Index_AbhiPrajapati.html Page Where i Have used Tailwind Css Scaling Property . Used it To make A ScreenShot for Readme File.

I made this index_AbhiPrajapati to Get a ScreenShot of Scaling Property For Readme File.
Attached a ScreenShot With Index_AbhiPrajapati.html File .
@TerrySmithDC
Copy link
Owner

Hi AbhiPrajapati, two main things:

  • In CONTRIBUTING.md there's instructions for setting up examples there, so your index file should not be in the root of the project.
  • Ideally the screen shot would be showing an example of the plugin, here you're using a mix of vanilla js and tailwind without adding the vertical rhythm plugin. So it's not really too useful in showing what to expect from the plugin. I've set up make commands you can use to develop with a tailwind environment, if you duplicate a example folder you should be able to quickly setup with what you need. Commands are in the CONTRIBUTING.md. :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants