Skip to content

[6.x] Inertia filesystems#18932

Draft
brianjhanson wants to merge 2 commits into
feature/inertia-uifrom
feature/inertia-filesystems
Draft

[6.x] Inertia filesystems#18932
brianjhanson wants to merge 2 commits into
feature/inertia-uifrom
feature/inertia-filesystems

Conversation

@brianjhanson
Copy link
Copy Markdown
Contributor

Description

Related issues

@brianjhanson brianjhanson changed the title Feature/inertia filesystems [6.x] Inertia filesystems May 20, 2026
<template>
<AppLayout>
<Pane appearance="raised">
<div class="grid gap-3">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It's probably good to have a <FormFields> wrapper that defines the spacing, just so it's consistent everywhere and if it has to change it's only in 1 file, or have <Pane> be a grid by default

@@ -0,0 +1,130 @@
<script setup lang="ts">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It might make sense to start putting pages in folders, for example pages/settings/filesystems/EditFilesystemPage.vue

Seb wrote a blog on how Spatie structures Inertia apps and there might be some ideas in there we could use as well.

For example if there are components needed just for filesystems we could put them in pages/settings/filesystems/components

name="url"
:required="true"
placeholder="//example.com/path/to/folder"
data-error-key="url"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we always have to provide this or could we fall back to what's provided in name for example and only need to provide this when it's different than the input name?

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