Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Login Page: Add quote and link to presentation website #261 #265

Merged
merged 7 commits into from
Sep 17, 2018
Merged

Login Page: Add quote and link to presentation website #261 #265

merged 7 commits into from
Sep 17, 2018

Conversation

christianeiselt
Copy link

close #261

@@ -12,6 +12,8 @@
<img src="/assets/images/registration/humanconnection.svg" alt="Human Connection"/>
</nuxt-link>
</div>
<p class="subtitle is-6">Viele kleine Leute, die in vielen kleinen Orten viele kleine Dinge tun, können das Gesicht der Welt verändern.</p>
<p class="subtitle is-7">Afrikanisches Sprichwort</p>
Copy link
Contributor

Choose a reason for hiding this comment

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

@christianeiselt please translate this quote

@appinteractive
Copy link
Member

appinteractive commented Sep 14, 2018

What about an English Version?

@christianeiselt
Copy link
Author

Deutsch: Viele kleine Leute, die in vielen kleinen Orten viele kleine Dinge tun, können das Gesicht der Welt verändern. - Afrikanisches Sprichwort
English: Many small people, who in many small places do many small things, that can alter the face of the world.
Mögliches Original: Viele kleine Leute, an vielen kleinen Orten, die viele kleine Dinge tun, werden das Antlitz dieser Welt verändern. - Sprichwort der Xhosa

locales/de.json Outdated
},
"quotes": {
"1": {
"quoteTxt": "Viele kleine Leute, die in vielen kleinen Orten viele kleine Dinge tun, können das Gesicht der Welt verändern.",
Copy link
Contributor

Choose a reason for hiding this comment

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

I love the Xhosa original!

Copy link
Contributor

Choose a reason for hiding this comment

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

I'd prefer that 😉

@@ -68,6 +68,8 @@
{{ $t('auth.login.label') }}
</hc-button>
</form>
<p class="subtitle is-6">{{ $t('quotes.1.quoteTxt') }}</p>
<p class="subtitle is-7">{{ $t('quotes.1.quoteAuthor') }}</p>
Copy link
Contributor

@roschaefer roschaefer Sep 14, 2018

Choose a reason for hiding this comment

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

@christianeiselt can we remove the 1? Just $t('quotes.quoteText')? Or if you want to add more quotes: $t('quotes.faceOfTheWorld.quoteTxt')?

Copy link
Author

@christianeiselt christianeiselt Sep 14, 2018

Choose a reason for hiding this comment

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

Is the goal to have only this one quote? Then $t('quotes.quoteText') will be good enough, of course.
I used the number because I thought the goal would be to display a random quote and I'd say it will be difficult to come up for a shortname for every quote... But the number is static and therefore still not good for manual inserts.

Also decreases the "padding-top" value of the quote author paragraph.
This needs attention as I only found a way by setting the style directly
to this div container.
<p class="subtitle is-7">{{ $t('quotes.african.author') }}</p>
<p class="subtitle is-6">
{{ $t('quotes.african.txt') }}
<div style="padding-top: 10px">
Copy link
Author

Choose a reason for hiding this comment

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

This is dirty... Can somebody point me to the right css class or where to create a new one for this purpose?
@roschaefer

Copy link
Member

Choose a reason for hiding this comment

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

Sure, just take a look at the other components. You can add a class name on the element and then style it inside the style tag on the component or page.

Copy link
Author

Choose a reason for hiding this comment

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

image
That is the current status.

Copy link
Author

@christianeiselt christianeiselt Sep 16, 2018

Choose a reason for hiding this comment

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

Sure, just take a look at the other components. You can add a class name on the element and then style it inside the style tag on the component or page.

Yeah, that's what I tried. But when I add .subtitle.is-8 as a copy of .subtitle.is-7 plus the additional padding-top: 10px; and change the class in the login.vue to subtitle is-8, it is not recognized and will be displayed with default style...

image

Actually, I just need to know, which stylesheet I need to manipulate, because there's so many of them...

Copy link
Member

Choose a reason for hiding this comment

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

Okay, I see that you are using div elements inside of p, that’s not valid. Try to use span instead.

And the Stylesheet is inside of the component you are workin on. Ignore everything inside the node modules folder. That are third party styles that not all are applied.

When you are searching for global styles, the are not .css but .scss files. The local styles live inside the .vue files then selve in an Style Tag. See https://vuejs.org/v2/guide/single-file-components.html

Copy link
Member

@appinteractive appinteractive Sep 16, 2018

Choose a reason for hiding this comment

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

I would stick with local styling in this case or see the https://bulma.io documentation for the available styles.

@roschaefer
Copy link
Contributor

@christianeiselt I think you can remove [WIP] from the title. This is very cool. I fixed small things and added a hint in the commit message.
@appinteractive this is good to go I would say. I find it odd that the text is below the login field. Can we at least move up the paragraph "Hier gehts zur Präsentationswebsite"?

@roschaefer
Copy link
Contributor

@appinteractive the footer looks terrible on mobile btw

@appinteractive
Copy link
Member

@roschaefer I have spoken with Sabine about that, I think we will redesign the login anyway in the direction like these: https://dribbble.com/shots/3829284-Dipnet-login-page

From the hierarchy, the first think should be the login form like before and after that the further info. On desktop we should put the infos on the site where we can add more info’s. But this should go into an separate issue.

@christianeiselt christianeiselt changed the title [WIP] Login Page: Add quote and link to presentation website #261 Login Page: Add quote and link to presentation website #261 Sep 17, 2018
@christianeiselt
Copy link
Author

Alright, I removed the WIP. Thanks for the input, y'all!

@roschaefer
Copy link
Contributor

@appinteractive would it be OK to merge this anyway? I bet, @christianeiselt would love to become a part of https://github.com/Human-Connection/WebApp/graphs/contributors

@appinteractive
Copy link
Member

Sure, would be glad to see another new face there! But could you open a follow up issue on for this? Or maybe even a project for login & registration?

@roschaefer
Copy link
Contributor

Ok, everybody, I hit "merge" now. I find it odd to have the text below but anyways.. 🤷‍♂️

@christianeiselt congrats to your first code contribution to Human Connection 🎉 🎈 🤗

@roschaefer roschaefer merged commit 54f6c59 into Human-Connection:develop Sep 17, 2018
@christianeiselt
Copy link
Author

christianeiselt commented Sep 17, 2018 via email

@Lulalaby
Copy link
Contributor

Lulalaby commented Sep 17, 2018

Well done @christianeiselt 😉.

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.

Login Page: Add quote and link to presentation website
4 participants