Skip to content

Add first version of main page text#116

Merged
Korosensei42 merged 4 commits intomasterfrom
kweirauch/configureMainPage
Aug 20, 2019
Merged

Add first version of main page text#116
Korosensei42 merged 4 commits intomasterfrom
kweirauch/configureMainPage

Conversation

@Korosensei42
Copy link
Copy Markdown
Contributor

This adresses #115, but might be changed due to further additions and the open questions in #115

Copy link
Copy Markdown
Member

@TimothyGillespie TimothyGillespie left a comment

Choose a reason for hiding this comment

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

I'd be happy with that. It's good.

Comment thread src/components/mainPage/mainPage.tsx Outdated
einen Beweis lösen soll. Will man eine eigene Aussage formulieren und beweisen,
kann man die Kategorie "Freies Üben" dafür nutzen. Wenn man noch nicht
allzu vertraut mit der Theorie hinter dem ist, was man beweisen möchte, findet
man im "Wiki" einige Informationen dazu. Bei Problemen befinden sich unter "Kontakt"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hier würde fehlen welche Probleme. Technische? Fachliche? Könnte man im Kontakt auch unterbringen.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Stimmt, das sollte da stehen. Da aber sowieso noch nicht ganz klar ist, wer da später unter Kontakt erwähnt werden soll, würde ich das auf einen späteren Zeitpunkt auslagern. Sollten wir nur im Hinterkopf behalten.

Comment thread src/components/mainPage/mainPage.tsx Outdated
return <div className={"addStyleHere"}>
<p>Willkommen auf unserer Startseite. Das hier sieht noch sehr nach Baustelle aus...</p>
<h1>Willkommen bei Diproche!</h1>
<p>"Diproche" steht für "Didactical proof checking". Das heißt im Wesentlichen, dass Diproche die
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

So ein tag der mehrere Zeilen beinhaltet ist glaube ich am lesbarsten untergebracht, wenn man diese Zeile noch in eine neue packt und dann ist alles eingeschobene der reine Text.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Das dürfte alles lesbarer machen. Da stimme ich dir zu.

Comment thread src/components/mainPage/mainPage.tsx Outdated
@@ -4,7 +4,17 @@ class ProofEditor extends React.Component {

public render() {
return <div className={"addStyleHere"}>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

className can be removed. I think this was inserted to just mark a possible stylesheet location, but it doesn't do anything as of now.
I might have even inserted it back then, but I am not sure anymore.

@TimothyGillespie
Copy link
Copy Markdown
Member

Feel free to still change the text though.

@Korosensei42
Copy link
Copy Markdown
Contributor Author

As the PR was approved before and I only changed minor things, I will merge now.

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