Fossil SCM
Increase min-height on the forum editor textareas a tick.
Commit
a15427ea99555aff3babe1671374b64edcdf35d395cb2c71f91cb7055b26cf7a
Parent
5e71becc09ec650…
2 files changed
+3
+2
-2
| --- src/fossil.page.forumpost.js | ||
| +++ src/fossil.page.forumpost.js | ||
| @@ -16,10 +16,13 @@ | ||
| 16 | 16 | unpredictably as the bottom boundary of the page moves. A weird |
| 17 | 17 | workaround (not invented here) is to add dummy blank padding to |
| 18 | 18 | the page to allow the preview widget to grow and shrink without |
| 19 | 19 | (usually) scrolling, but whether it does so really depends on its |
| 20 | 20 | size. |
| 21 | + | |
| 22 | + We could maybe get the same effect by adding this size as | |
| 23 | + padding-bottom to document.body instead of as a new element. | |
| 21 | 24 | */ |
| 22 | 25 | const dummyPadding = D.div(); |
| 23 | 26 | dummyPadding.style.height = '75em'; |
| 24 | 27 | /* Keep track of ForumPostEditor instances so we can remove this |
| 25 | 28 | padding when none are active. */ |
| 26 | 29 |
| --- src/fossil.page.forumpost.js | |
| +++ src/fossil.page.forumpost.js | |
| @@ -16,10 +16,13 @@ | |
| 16 | unpredictably as the bottom boundary of the page moves. A weird |
| 17 | workaround (not invented here) is to add dummy blank padding to |
| 18 | the page to allow the preview widget to grow and shrink without |
| 19 | (usually) scrolling, but whether it does so really depends on its |
| 20 | size. |
| 21 | */ |
| 22 | const dummyPadding = D.div(); |
| 23 | dummyPadding.style.height = '75em'; |
| 24 | /* Keep track of ForumPostEditor instances so we can remove this |
| 25 | padding when none are active. */ |
| 26 |
| --- src/fossil.page.forumpost.js | |
| +++ src/fossil.page.forumpost.js | |
| @@ -16,10 +16,13 @@ | |
| 16 | unpredictably as the bottom boundary of the page moves. A weird |
| 17 | workaround (not invented here) is to add dummy blank padding to |
| 18 | the page to allow the preview widget to grow and shrink without |
| 19 | (usually) scrolling, but whether it does so really depends on its |
| 20 | size. |
| 21 | |
| 22 | We could maybe get the same effect by adding this size as |
| 23 | padding-bottom to document.body instead of as a new element. |
| 24 | */ |
| 25 | const dummyPadding = D.div(); |
| 26 | dummyPadding.style.height = '75em'; |
| 27 | /* Keep track of ForumPostEditor instances so we can remove this |
| 28 | padding when none are active. */ |
| 29 |
+2
-2
| --- src/style.forum.css | ||
| +++ src/style.forum.css | ||
| @@ -48,15 +48,15 @@ | ||
| 48 | 48 | flex-direction: column; |
| 49 | 49 | } |
| 50 | 50 | |
| 51 | 51 | .ForumPostEditor .tab-panel.editor-wrapper > .editor { |
| 52 | 52 | max-width: initial; |
| 53 | - min-height: 14em; | |
| 53 | + min-height: 20em; | |
| 54 | 54 | flex-grow: 1 |
| 55 | 55 | } |
| 56 | 56 | body.cpage-forumnew .ForumPostEditor .tab-panel.editor-wrapper > .editor { |
| 57 | - min-height: 24em; | |
| 57 | + min-height: 28em; | |
| 58 | 58 | } |
| 59 | 59 | |
| 60 | 60 | .ForumPostEditor > .buttons > .mimetype-wrapper, |
| 61 | 61 | .ForumPostEditor > .titlebar{ |
| 62 | 62 | display: flex; |
| 63 | 63 |
| --- src/style.forum.css | |
| +++ src/style.forum.css | |
| @@ -48,15 +48,15 @@ | |
| 48 | flex-direction: column; |
| 49 | } |
| 50 | |
| 51 | .ForumPostEditor .tab-panel.editor-wrapper > .editor { |
| 52 | max-width: initial; |
| 53 | min-height: 14em; |
| 54 | flex-grow: 1 |
| 55 | } |
| 56 | body.cpage-forumnew .ForumPostEditor .tab-panel.editor-wrapper > .editor { |
| 57 | min-height: 24em; |
| 58 | } |
| 59 | |
| 60 | .ForumPostEditor > .buttons > .mimetype-wrapper, |
| 61 | .ForumPostEditor > .titlebar{ |
| 62 | display: flex; |
| 63 |
| --- src/style.forum.css | |
| +++ src/style.forum.css | |
| @@ -48,15 +48,15 @@ | |
| 48 | flex-direction: column; |
| 49 | } |
| 50 | |
| 51 | .ForumPostEditor .tab-panel.editor-wrapper > .editor { |
| 52 | max-width: initial; |
| 53 | min-height: 20em; |
| 54 | flex-grow: 1 |
| 55 | } |
| 56 | body.cpage-forumnew .ForumPostEditor .tab-panel.editor-wrapper > .editor { |
| 57 | min-height: 28em; |
| 58 | } |
| 59 | |
| 60 | .ForumPostEditor > .buttons > .mimetype-wrapper, |
| 61 | .ForumPostEditor > .titlebar{ |
| 62 | display: flex; |
| 63 |