Skip to content

Commit bcdf592

Browse files
authored
Merge pull request #1089 from jendave/main
Widen and lengthen Rollable Table rich text editor
2 parents 3cc3425 + 0380ba3 commit bcdf592

2 files changed

Lines changed: 35 additions & 1 deletion

File tree

CHANGELOG.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,8 @@
22

33
## Next Release
44

5-
- Fresh import from datasworn-community data
5+
- Fresh import from datasworn-community data.
6+
- Widen and lengthen Rollable Table rich text editor. Editing icons no longer wrap around and text box can show multiple lines.
67

78
## 1.28.0
89

src/styles/editor.less

Lines changed: 33 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -86,3 +86,36 @@
8686
padding: var(--ironsworn-spacer-lg);
8787
}
8888
}
89+
90+
// The core TableResult config sheet (opened when editing an individual roll
91+
// table row) embeds a ProseMirror editor for the result Description. Its default
92+
// 560px width is too narrow, so the editor toolbar buttons wrap onto a second
93+
// row. This is a core Foundry sheet and never gets the `.ironsworn` class, so
94+
// widen it directly. AppV2 applies `width` as an inline style; a stylesheet
95+
// `min-width` wins over it and forces the sheet wide enough for the toolbar.
96+
.table-result-config {
97+
min-width: 640px;
98+
99+
// The Description editor collapses to a single line here. Core already sizes
100+
// the `<prose-mirror>` element (it's a 200px flex column), but its inner
101+
// `.editor-content` only gets `height: 100%` from a rule scoped to
102+
// `body.game .sheet.app` — the V1 frame class. This is an AppV2 sheet
103+
// (`.application`, not `.app`), so the content never fills the column and
104+
// sits one line tall at the top. Make it grow to fill the available height.
105+
// The `<prose-mirror>` wrapper also carries the `.editor` class. On v13,
106+
// `phosphor.scss` re-emits `editor.scss` under `.ironcolor__phosphor.foundry-v13`
107+
// (via `@include fvtt.ui`), producing `.ironcolor__phosphor.foundry-v13 .editor
108+
// { min-height: 100px }` — three classes, which outranks a bare
109+
// `.table-result-config prose-mirror` (two classes + element) and caps the
110+
// editor at 100px. (v14 skips that include, so this only bites on v13.) Match
111+
// `.editor` here too so the selector wins on both versions.
112+
prose-mirror.editor {
113+
min-height: 240px;
114+
}
115+
116+
.editor-content {
117+
flex: 1;
118+
height: 100%;
119+
overflow-y: auto;
120+
}
121+
}

0 commit comments

Comments
 (0)