LM-150 Add collapse animation #17
Conversation
rm--
force-pushed
the
LM-150_collapse-with-vue-collapsed
branch
from
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.
- 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.
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.
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.
Heißen jetzt details.
| :source="expanded ? mdFullText : mdAbstract" | ||
| /> | ||
|
|
||
| <div class="description-column__markdown"> |
There was a problem hiding this comment.
Durch das Anfügen des zweiten Teils an das "abstract" entsteht hier eine ungewünschte newline.
There was a problem hiding this comment.
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. |
dweiss96
force-pushed
the
LM-150_collapse-with-vue-collapsed
branch
from
f63e666
to
7d09cdb
Compare
so ist es es imo perfekt. |
As alternative idea to self coding the animation as tried in #14.