Skip to content

Commit 445af2a

Browse files
committed
chore: fix order in contributing guide
1 parent 2414cc0 commit 445af2a

File tree

1 file changed

+16
-16
lines changed

1 file changed

+16
-16
lines changed

CONTRIBUTING.md

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -357,22 +357,6 @@ Query parameters work as expected:
357357
<NuxtLink :to="{ name: 'compare', query: { packages: pkg.name } }">Compare</NuxtLink>
358358
```
359359

360-
### Cursor and navigation
361-
362-
We prefer consistency in user everyday experience
363-
364-
npmx uses `cursor-pointer` only for links. For all other elements, including buttons, use the default cursor (_or other appropriate cursors to show state_).
365-
366-
**What counts as a link**
367-
368-
A link is any element that change or show meaningful content:
369-
370-
- **Not meaningful content:** burger menu, "likes" button - the user does not get substantive information from them.
371-
- **Meaningful content:** a dialog with a chart, auth button, tabs with new data—they convey new or important information.
372-
373-
> [!IMPORTANT]
374-
> Always prefer implementing such changes as real links so they can be opened in a new tab, shared or reloaded, and so the same content is available at a stable URL.
375-
376360
#### Package routes
377361

378362
For package links, use the auto-imported `packageRoute()` utility from `app/utils/router.ts`. It handles scoped/unscoped packages and optional versions:
@@ -409,6 +393,22 @@ For package links, use the auto-imported `packageRoute()` utility from `app/util
409393
| `~username` | `/~:username` | `username` |
410394
| `~username-orgs` | `/~:username/orgs` | `username` |
411395

396+
### Cursor and navigation
397+
398+
We prefer consistency in user everyday experience
399+
400+
npmx uses `cursor-pointer` only for links. For all other elements, including buttons, use the default cursor (_or other appropriate cursors to show state_).
401+
402+
**What counts as a link**
403+
404+
A link is any element that change or show meaningful content:
405+
406+
- **Not meaningful content:** burger menu, "likes" button - the user does not get substantive information from them.
407+
- **Meaningful content:** a dialog with a chart, auth button, tabs with new data—they convey new or important information.
408+
409+
> [!IMPORTANT]
410+
> Always prefer implementing such changes as real links so they can be opened in a new tab, shared or reloaded, and so the same content is available at a stable URL.
411+
412412
## RTL Support
413413

414414
We support `right-to-left` languages, we need to make sure that the UI is working correctly in both directions.

0 commit comments

Comments
 (0)