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

Update documentation for all @compiled/ packages #1689

Merged
merged 7 commits into from
Jul 17, 2024

Conversation

dddlr
Copy link
Collaborator

@dddlr dddlr commented Jul 15, 2024

Follow-up to #1687 and #1688

Click this link to see how the website looks: https://deploy-preview-1689--compiled-css-in-js.netlify.app/

Main change:

Below are the other changes I've made. I recommend looking at the commit-by-commit:

  • Update "Atomic CSS" page to use object styles, not tagged template strings
  • Update @compiled/react documentation to explicitly deprecate styled and ClassName APIs, and make css/cssMap/keyframes more prominent
  • Update fonts and font sizes to fit more text on a page (and because Roboto belongs in the early 2010s imo)
    • This is a quick hack to make it more readable; I'm not a professional designer, so it's not the most bang-up job. Please be nice 😆
    • This also substantially helps readability for the other changes in this PR
    • If we have more time, we can consider (1) migrating the docs to a centralised platform or (2) redesigning the website entirely.
  • Change the "Installation" page to recommend Parcel over Webpack
  • Update title of 'Media queries and other at rules' to more easily fit the sidebar

Copy link

changeset-bot bot commented Jul 15, 2024

⚠️ No Changeset found

Latest commit: cdf4be7

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

Copy link

netlify bot commented Jul 15, 2024

Deploy Preview for compiled-css-in-js canceled.

Name Link
🔨 Latest commit cdf4be7
🔍 Latest deploy log https://app.netlify.com/sites/compiled-css-in-js/deploys/66973fecdad94c00089f7699

@dddlr dddlr changed the title Chore/docs revamp UI styling standard Update documentation for all @compiled/ packages Jul 15, 2024

#### sortAtRules

Unused.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I reckon dealing with stuff that's unused, deprecated, obsolete etc would be out of scope of this PR - as a website/docs-focussed PR, I want to avoid making changes to the Compiled packages

I've made an issue so that we can properly clean these up in the future #1690


#### processXcss

An option we are only using internally. Don't use!
Copy link
Collaborator Author

@dddlr dddlr Jul 15, 2024

Choose a reason for hiding this comment

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

not sure if there's any point documenting internal options... i've erred towards including them just so we can use the docs as a single source of truth

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think that's fine to be honest, probably helps the internal team as well, eh?

@@ -32,4 +32,4 @@ export default [

## Rules

See [here](https://github.com/atlassian-labs/compiled/tree/master/packages/eslint-plugin) for a list of rules in `@compiled/eslint-plugin`.
See [here](https://github.com/atlassian-labs/compiled/tree/master/packages/eslint-plugin) for a non-exhaustive list of rules in `@compiled/eslint-plugin`.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Tracked through #1691


Enables baking your styles into JavaScript.
- Type: `string[]`
- Default: none, but note that `enhanced-resolver` and `@compiled/babel-plugin` have their own different defaults when the `extensions` option is not set.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The fact that there isn't a single default being applied everywhere does admittedly sound like a bug

enhanced-resolver uses ['js', 'json', 'node'] as a default, while @compiled/babel-plugin uses ['.js', '.jsx', '.ts', '.tsx'] as a default - I made an issue to track it here #1692


#### processXcss

An option we are only using internally. Don't use!
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think that's fine to be honest, probably helps the internal team as well, eh?

liamqma
liamqma previously approved these changes Jul 16, 2024
@dddlr dddlr force-pushed the chore/docs-revamp-ui-styling-standard branch from fcd709e to cdf4be7 Compare July 17, 2024 03:52
@dddlr dddlr merged commit 18024be into master Jul 17, 2024
13 checks passed
@dddlr dddlr deleted the chore/docs-revamp-ui-styling-standard branch July 17, 2024 04:06
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