-
Notifications
You must be signed in to change notification settings - Fork 1
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
LM-150 Add collapse animation #17
Conversation
ef4ca21
to
f63e666
Compare
|
src/components/about-row.vue
Outdated
<vue-markdown | ||
v-if="props.markdown" | ||
class="about-row__markdown__content" | ||
:source="mdExplenation" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- typo: explanation
- Aber ich denke wir sollten ein besseres Wort für den Inhalt finden, das überall passt.
content
,details
,body
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Als content
bzw. body
würde ich eher beide Texte zusammen sehen. details
kann man nehmen, obwohl es auch nicht immer passt. Bei dem Text zur Idee ist es mehr eine "Einschränkung" bzw. erweiterte Ausführung zum "Abstract". details
passt gefühlt auch nur wirklich bei der Ferry Porsche Challenge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bei Accordions ist es wohl meist title und content oder key und content. Ich dachte man kann sich daran orientieren.
Oder man gibt ihnen Zahlen, Paragraph1 und Paragraph2.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Heißen jetzt details
.
:source="expanded ? mdFullText : mdAbstract" | ||
/> | ||
|
||
<div class="description-column__markdown"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Durch das Anfügen des zweiten Teils an das "abstract" entsteht hier eine ungewünschte newline.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed. Hab vergessen, dass dort auch wieder Standard-HTML Abstände gesetzt werden.
Das passiert aktuell auch schon. Hab jetzt feste Werte hinterlegt, die aus den bisherigen Dimensionen entnommen sind. |
f63e666
to
7d09cdb
Compare
so ist es es imo perfekt. |
As alternative idea to self coding the animation as tried in #14.