From c1d7443c872b3f70b8b2903110e104ca4117cb8d Mon Sep 17 00:00:00 2001 From: Scott Tolksdorf Date: Sat, 14 May 2016 13:08:27 -0400 Subject: [PATCH] Adding a source route instead of jsut opening a new window --- client/homebrew/pages/sharePage/sharePage.jsx | 14 +------------- client/main/main.jsx | 2 +- server/homebrew.server.js | 12 ++++++++++++ shared/naturalcrit/nav/nav.jsx | 9 +++------ todo.md | 2 +- 5 files changed, 18 insertions(+), 21 deletions(-) diff --git a/client/homebrew/pages/sharePage/sharePage.jsx b/client/homebrew/pages/sharePage/sharePage.jsx index 04c7ff875..7252c5227 100644 --- a/client/homebrew/pages/sharePage/sharePage.jsx +++ b/client/homebrew/pages/sharePage/sharePage.jsx @@ -9,10 +9,6 @@ var PrintLink = require('../../navbar/print.navitem.jsx'); var BrewRenderer = require('../../brewRenderer/brewRenderer.jsx'); -var replaceAll = function(str, find, replace) { - return str.replace(new RegExp(find, 'g'), replace); -} - var SharePage = React.createClass({ getDefaultProps: function() { return { @@ -27,14 +23,6 @@ var SharePage = React.createClass({ }; }, - openSourceWindow : function(){ - var sourceWindow = window.open(); - var content = replaceAll(this.props.brew.text, '<', '<'); - content = replaceAll(content, '>', '>'); - sourceWindow.document.write('
' + content + '
'); - }, - - render : function(){ return
@@ -44,7 +32,7 @@ var SharePage = React.createClass({ - + source diff --git a/client/main/main.jsx b/client/main/main.jsx index 94d98eae3..815d2c73e 100644 --- a/client/main/main.jsx +++ b/client/main/main.jsx @@ -28,7 +28,7 @@ var Main = React.createClass({ icon : , desc : 'Make authentic-looking 5e homebrews using Markdown', - show : true, + show : false, beta : true }, { diff --git a/server/homebrew.server.js b/server/homebrew.server.js index 727212c4c..f37032967 100644 --- a/server/homebrew.server.js +++ b/server/homebrew.server.js @@ -97,6 +97,18 @@ module.exports = function(app){ }); }); + //Source page + String.prototype.replaceAll = function(s,r){return this.split(s).join(r)} + app.get('/homebrew/source/:id', function(req, res){ + HomebrewModel.find({shareId : req.params.id}, function(err, objs){ + if(err || !objs.length) return res.status(404).send('Could not find Homebrew with that id'); + var brew = null; + if(objs.length) brew = objs[0]; + var text = brew.text.replaceAll('<', '<').replaceAll('>', '>'); + return res.send(`
${text}
`); + }); + }); + //Home and 404, etc. var welcomeText = require('fs').readFileSync('./client/homebrew/pages/homePage/welcome_msg.txt', 'utf8'); var changelogText = require('fs').readFileSync('./changelog.md', 'utf8'); diff --git a/shared/naturalcrit/nav/nav.jsx b/shared/naturalcrit/nav/nav.jsx index c696dbfd6..b082a85ed 100644 --- a/shared/naturalcrit/nav/nav.jsx +++ b/shared/naturalcrit/nav/nav.jsx @@ -2,6 +2,8 @@ var React = require('react'); var _ = require('lodash'); var cx = require('classnames'); +var NaturalCritIcon = require('naturalcrit/svg/naturalcrit.svg.jsx'); + var Nav = { base : React.createClass({ render : function(){ @@ -14,17 +16,12 @@ var Nav = { }), logo : function(){ return - {Nav.logoSVG()} + NaturalCrit ; }, - logoSVG : function(){ - return - - ; - }, section : React.createClass({ render : function(){ diff --git a/todo.md b/todo.md index dc4b58e47..286f8408d 100644 --- a/todo.md +++ b/todo.md @@ -23,7 +23,7 @@ X Move snippets into their new groups - add /source/shareId route - Add stats nav item - make both ids indexes in mongoose -- Fix main Page +X Fix main Page - Write new welcome message ## v1.6