Skip to content
This repository has been archived by the owner on Mar 25, 2024. It is now read-only.

Søke loupe-fungerer ikke på desktop #212

Open
truls1502 opened this issue Aug 28, 2018 · 2 comments
Open

Søke loupe-fungerer ikke på desktop #212

truls1502 opened this issue Aug 28, 2018 · 2 comments

Comments

@truls1502
Copy link
Member

Problemstilling

Jeg oppdaget det problemet i OS 1.18, og er usikker om det var der før som i 1.17. Uansett er problemet når man trykker på Søke-loupe ikonet - der kom sånn animasjon, men for å søke funker det ikke.

Løsning/forslag

Jeg er litt usikker hvor årsaken ligger, men jeg tenke tror kanskje det er noe themeing som skaper noe feil her.

Fint om dere kan hjelpe til å feilsøke det problemet! :)

@truls1502
Copy link
Member Author

Svar på feilsøket

Jeg tror vi har funnet feilen etter fått hjelp fra @Kingdutch siden han nevte det har skjedd en endring fra https://www.drupal.org/project/social/releases/8.x-1.5 som står:

DS-4122 - update search block interaction in navbar
The 3rd party javascript is removed from the socialbase theme. The original search block that was in region hidden and included in the menu via a preprocess function is now moved to content_top region. This block is still loaded in the menu and shown on the mobile menu. For desktop there is a trigger button that toggles the search block in the content _top region. It needs to be in a separate region because it needs to be on top of all other content (z-index).
Manual action:
In your subtheme move the Search content block to the content top region.

Nytt problemstilling

Per i dag kan man ikke arve blokker fra base theme - det står der https://www.drupal.org/docs/8/theming/sub-theme-inheritance - uansett sto man kunne gjøre ved å legge en hook, jeg har prøvd den men uten hell:

/**
 * Implements hook_theme_suggestions_HOOK_alter for blocks.
 */
function drupalnorge_theme_theme_suggestions_block_alter(&$suggestions, $variables) {

  // Load theme suggestions for blocks from parent theme.
  foreach ($suggestions as &$suggestion) {
    $suggestion = str_replace('drupalnorge_theme_', 'socialblue_', $suggestion);
  }
}

API: https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Render%21theme.api.php/function/hook_theme_suggestions_alter/

Jeg prøver å leie en omvei som vi slipper å kopiere filene inn i vårt siden vår theme skal være enkelt og minimalt filer slik at det er letter ved oppdatering til nyere OS-versjoner i fremtiden. ;)

Hvis noen vet omvei eller ser noe tydelig feil skrevet i mitt forslag, så setter jeg pris på tilbakemelding 🤓

@truls1502
Copy link
Member Author

Jeg har postet problemet på https://www.drupal.org/project/social/issues/3004361

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant