TMS-1033: Add new optional hero-museum component #505
Conversation
| @@ -0,0 +1,17 @@ | |||
| .hero-museum { | |||
| height: auto !important; | |||
There was a problem hiding this comment.
| height: auto !important; | |
| height: auto !important; | |
|
|
||
| h1 { | ||
| @include from($desktop) { | ||
| font-size: 4.41rem !important; // sass-lint:disable-line no-important |
There was a problem hiding this comment.
Pitääkö olla näin spesifi?
There was a problem hiding this comment.
Tai voisko tän laittaa johonkin muuttujaan edes?
There was a problem hiding this comment.
Tää hero on otettu museosivustojen teemasta, ilmeisesti joskus on päädytty näin spesifiin kokoon ihan tarkoituksella 😅 Muualla tätä kokoa ei ookaan käytössä
| @@ -0,0 +1,222 @@ | |||
| <?php | |||
There was a problem hiding this comment.
| <?php | |
| <?php | |
lib/ACF/Fields/HeroMuseumFields.php
Outdated
| * @return array | ||
| * @throws Exception In case of invalid ACF option. | ||
| */ | ||
| public function get_hero_group_fields( string $key, string $group, array $strings ) : array { |
There was a problem hiding this comment.
Tää vois varmaan olla protected?
| @@ -85,21 +89,28 @@ public function sub_fields( $key ) : void { | |||
| ); | |||
|
|
|||
| $color_theme_select = ( new Field\Select( $this->strings['color_selection']['title'] ) ) | |||
| ->set_key( $key . '_theme_color' ) | |||
| ->set_key( '{$key}_theme_color' ) | |||
There was a problem hiding this comment.
Tarvii tupla-quotet jos interpoloi stringejä, muuten tää on ihan literal '{$key}_theme_color' 😬
There was a problem hiding this comment.
| ->set_key( '{$key}_theme_color' ) | |
| ->set_key( "{$key}_theme_color" ) |
There was a problem hiding this comment.
tsiisus :D mitenhän tää on edes toiminu tälläsenä testatessa
| @@ -0,0 +1,70 @@ | |||
| <?php | |||
There was a problem hiding this comment.
| <?php | |
| <?php | |
| /** | ||
| * Define formatter name | ||
| */ | ||
| const NAME = 'HeroMuseum'; |
There was a problem hiding this comment.
Käytetäänkö näitä nameja jossain, ja pitääkö tän olla PascalCasella, vai voisko olla hero-museum tms?
There was a problem hiding this comment.
Samalla mallilla tehty kuin muissakin formattereissa, tätä ei taideta käyttää muualla
| * Hooks | ||
| */ | ||
| public function hooks() : void { | ||
| add_filter( |
There was a problem hiding this comment.
| add_filter( | |
| \add_filter( |
| @@ -0,0 +1,66 @@ | |||
| <section {?anchor}id="{anchor|attr}"{/anchor} | |||
| class="hero-museum"> | |||
There was a problem hiding this comment.
Ehkä vähän turha rivinvaihto
|
|
||
| {?link} | ||
| <div class="button-container"> | ||
| {>"ui/button-link" classes=button_classes icon="chevron-right" icon_classes="icon--medium" /} |
There was a problem hiding this comment.
Pitäiskö tähänki lisätä link=link, kun tuolla alempanakin on? Toimii kyllä ilmankin, mutta olis vähän verbosempi.
Severa-ID: 2108
Severa-kuvaus: TMS-1033 Taidevaltakunta-sivustolle omanlainen heroelementti
Task: https://hiondigital.atlassian.net/browse/TMS-1033
Description
Added museum-theme style hero as an optional component
Types of changes