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

Use Tailwind for basic styling #14

Merged
merged 1 commit into from
Sep 9, 2024
Merged

Conversation

danlivings-dxw
Copy link
Contributor

Tailwind is a CSS framework designed to provide a collection of composable classes that can be combined to style elements in a visually coherent way.

Typically, Tailwind is used a devDependency and built to ensure a minimal amount of CSS is used. However, in the spirit of being as straightforward to maintain as possible, instead this just includes the Tailwind CDN script in the index.njk template.

The visual changes include:

  • Making the 'Towtruck' title prominent in a header bar
  • A card-style interface showing the table of repositories for each GitHub installation
  • A basic table layout with alternating row colours for easier visual identification.

Below is a preview of these changes:
image

Copy link
Contributor

@richpjames richpjames left a comment

Choose a reason for hiding this comment

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

I agree with the approach of just importing Tailwind into the template here.
I think in time it would be good to use BEM styling similar to that of the Playbook (some shared dxw stylesheets would be useful) but I agree that the approach here is the best for now in order for us to progress quickly.

@danlivings-dxw danlivings-dxw force-pushed the feature/tailwind-css branch 4 times, most recently from c919d1e to c1cae09 Compare September 9, 2024 13:48
[Tailwind](https://tailwindcss.com/) is a CSS framework designed to
provide a collection of composable classes that can be combined to style
elements in a visually coherent way.

Typically, Tailwind is used a `devDependency` and built to ensure a
minimal amount of CSS is used. However, in the spirit of being as
straightforward to maintain as possible, instead this just includes the
Tailwind CDN script in the `index.njk` template.

The visual changes include:
- Making the 'Towtruck' title prominent in a header bar
- A card-style interface showing the table of repositories for each GitHub installation
- A basic table layout with alternating row colours for easier visual
  identification.
@danlivings-dxw danlivings-dxw merged commit 74c8973 into main Sep 9, 2024
2 checks passed
@danlivings-dxw danlivings-dxw deleted the feature/tailwind-css branch September 9, 2024 13:53
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.

2 participants