f.weber/homebrewery
0
0
Fork 0
mirror of https://github.com/naturalcrit/homebrewery.git synced 2025-12-24 16:22:44 +00:00
Code Wiki Activity

Fix /page not working in legacy mode. (#1233)

Browse Source 
* Fix brew styles overwriting each other.

* Word wrapping, start fixing spacing on Title letter

* Fix \page in legacy brews when not at line start
This commit is contained in:
Trevor Buckner
2021-02-08 13:58:24 -05:00
committed by GitHub
parent c642a35fb3
commit e5ccfa3a50
4 changed files with 53 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files

34
client/homebrew/brewRenderer/brewRenderer.jsx
View File

@@ -20,15 +20,17 @@ const BrewRenderer = createClass({
getDefaultProps : function() {
return {
text : '',
renderer : '',
renderer : 'legacy',
errors : []
};
},
getInitialState : function() {
const pages = this.props.text.split(/^\\page/gm);
let renderer = 'legacy';
if(this.props.renderer)
renderer = this.props.renderer;
let pages;
if(this.props.renderer == 'legacy') {
pages = this.props.text.split('\\page');
} else {
pages = this.props.text.split(/^\\page/gm);
}
return {
viewablePageNumber : 0,
@@ -38,7 +40,6 @@ const BrewRenderer = createClass({
pages : pages,
usePPR : pages.length >= PPR_THRESHOLD,
visibility : 'hidden',
renderer : renderer,
initialContent : `<!DOCTYPE html><html><head>
<link href="//use.fontawesome.com/releases/v5.15.1/css/all.css" rel="stylesheet" />
<link href="//fonts.googleapis.com/css?family=Open+Sans:400,300,600,700" rel="stylesheet" type="text/css" />
@@ -54,12 +55,19 @@ const BrewRenderer = createClass({
window.removeEventListener('resize', this.updateSize);
},
componentWillReceiveProps : function(nextProps) {
const pages = nextProps.text.split(/^\\page/gm);
this.setState({
pages : pages,
usePPR : pages.length >= PPR_THRESHOLD
});
componentDidUpdate : function(prevProps) {
if(prevProps.text !== this.props.text) {
let pages;
if(this.props.renderer == 'legacy') {
pages = this.props.text.split('\\page');
} else {
pages = this.props.text.split(/^\\page/gm);
}
this.setState({
pages : pages,
usePPR : pages.length >= PPR_THRESHOLD
});
}
},
updateSize : function() {
@@ -114,7 +122,7 @@ const BrewRenderer = createClass({
},
renderPage : function(pageText, index){
if(this.state.renderer == 'legacy')
if(this.props.renderer == 'legacy')
return <div className='phb' id={`p${index + 1}`} dangerouslySetInnerHTML={{ __html: MarkdownLegacy.render(pageText) }} key={index} />;
else
return <div className='phb3' id={`p${index + 1}`} dangerouslySetInnerHTML={{ __html: Markdown.render(pageText) }} key={index} />;

16
client/homebrew/editor/editor.jsx
View File

@@ -24,7 +24,7 @@ const Editor = createClass({
metadata : {},
onMetadataChange : ()=>{},
showMetaButton : true,
renderer : ''
renderer : 'legacy'
};
},
getInitialState : function() {
@@ -93,13 +93,21 @@ const Editor = createClass({
codeMirror.removeLineClass(lineNumber, 'background');
codeMirror.removeLineClass(lineNumber, 'text');
if(line.startsWith('\\page')){
codeMirror.addLineClass(lineNumber, 'background', 'pageLine');
r.push(lineNumber);
// Legacy Codemirror styling
if(this.props.renderer == 'legacy') {
if(line.includes('\\page')){
codeMirror.addLineClass(lineNumber, 'background', 'pageLine');
r.push(lineNumber);
}
}
// New Codemirror styling for V3 renderer
if(this.props.renderer == 'V3') {
if(line.startsWith('\\page')){
codeMirror.addLineClass(lineNumber, 'background', 'pageLine');
r.push(lineNumber);
}
if(line.startsWith('\\column')){
codeMirror.addLineClass(lineNumber, 'text', 'columnSplit');
r.push(lineNumber);

24
client/homebrew/phbStyle/phb.style.less
View File

@@ -1,6 +1,6 @@
@import (less) './client/homebrew/phbStyle/phb.fonts.css';
@import (less) './client/homebrew/phbStyle/phb.assets.less';
@import (less) './client/homebrew/phbStyle/phb.depricated.less';
//Colors
@background : #EEE5CE;
@noteGreen : #e0e5c1;
@@ -17,12 +17,11 @@ body {
}
.useSansSerif(){
font-family : ScalySans;
font-size : 10pt;
em{
font-family : ScalySans;
font-style : italic;
}
strong{
font-family : ScalySans;
font-weight : 800;
letter-spacing : -0.02em;
}
@@ -48,8 +47,7 @@ body {
overflow : hidden;
height : 279.4mm;
width : 215.9mm;
padding : 1.0cm 1.7cm;
padding-bottom : 1.5cm;
padding : 1.0cm 1.7cm, 1.5cm;
background-color : @background;
background-image : @backgroundImage;
font-family : BookSanity;
@@ -61,6 +59,7 @@ body {
// * BASE
// *****************************/
p{
overflow-wrap : break-word;
padding-bottom : 0.8em;
line-height : 1.3em;
&+p{
@@ -123,9 +122,20 @@ body {
&+p::first-letter{
float : left;
font-family : Solberry;
font-size : 10em;
color : #222;
line-height : 0.8em;
font-size: 3.1cm;
padding-top: 4px;
padding-bottom: 4px;
margin-top: -6px;
margin-left: -6px;
background-image: linear-gradient(-45deg, #322814, #998250, #322814);
background-clip: text;
-webkit-background-clip: text;
color: rgba(0, 0, 0, 0);
}
&+p::first-line{
font-size : .385cm;
font-variant : small-caps;
}
}
h2{

6
shared/naturalcrit/codeEditor/codeEditor.jsx
View File

@@ -61,9 +61,9 @@ const CodeEditor = createClass({
}
},
componentWillReceiveProps : function(nextProps){
if(this.codeMirror && nextProps.value !== undefined && this.codeMirror.getValue() != nextProps.value) {
this.codeMirror.setValue(nextProps.value);
componentDidUpdate : function(prevProps) {
if(this.codeMirror && this.props.value !== undefined && this.codeMirror.getValue() != this.props.value) {
this.codeMirror.setValue(this.props.value);
}
},