Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

[OPENY-1375] Search icon and form removed from Carnation theme 8.x 2.x #1384

Merged
merged 5 commits into from
Dec 13, 2018
Merged

[OPENY-1375] Search icon and form removed from Carnation theme 8.x 2.x #1384

merged 5 commits into from
Dec 13, 2018

Conversation

afi13
Copy link
Contributor

@afi13 afi13 commented Nov 14, 2018

Issue #1375

Steps for review for new installations

  • Install Carnation theme and set as the default theme.
  • Go to /admin/appearance/settings/openy_carnation and make sure 'Display the search form' setting is enabled by default.
  • Make sure search icon anf form are displayed for desktop and mobile.
  • Go to /admin/appearance/settings/openy_carnation and disable 'Display the search form' .
  • Make sure search icon and form are hidden for desktops.
  • Make sure search icon and form are hidden for mobile.
  • There no any visible FE regressions.

Steps for review of upgrade path

  • Make sure 'Display the search form' setting is enabled after db updates.
  • Make sure search icon and form are displayed for desktop and mobile.
  • Go to /admin/appearance/settings/openy_carnation and disable 'Display the search form' .
  • Make sure search icon and form are hidden for desktops.
  • Make sure search icon and form are hidden for mobile.
  • There no any visible FE regressions.

General checks

  • All coding styles are fulfilled and there are no any issues reported by CodeSniffer CI.
    CI code sniffer errors
  • All tests are running and there are no failed tests reported by CI.
    Behat test results
  • Documentation has been updated according to PR changes.
  • Steps for review have been provided according to PR changes.
    Steps for review
  • Make sure you've provided all necessary hook_update_N to support upgrade path.
  • Make sure your git email is associated with account on drupal.org, otherwise you won't get commits there.
    drupal.org email
  • If you would like to get credits on drupal.org, check documentation.

Thank you for your contribution!

@afi13 afi13 self-assigned this Nov 14, 2018
@afi13 afi13 requested a review from ddrozdik November 14, 2018 11:47
@afi13 afi13 added PR: Needs Review Needs someone review ( code ) PR: Needs Testing Manual testing is needed labels Nov 14, 2018
@ddrozdik ddrozdik requested a review from podarok November 14, 2018 14:29
@ddrozdik ddrozdik added PR: Code Reviewed Code reviewed. Ready for QA and removed PR: Needs Review Needs someone review ( code ) labels Nov 14, 2018
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 14, 2018
@ddrozdik ddrozdik added this to the OpenY 2.0 milestone Nov 14, 2018
@ddrozdik ddrozdik changed the title Openy 1375 carnation hide search 8.x 2.x [OPENY-1375] Search icon and form removed from Carnation theme 8.x 2.x Nov 14, 2018
@ymcatwincities ymcatwincities deleted a comment from gundevel Dec 11, 2018
@ymcatwincities ymcatwincities deleted a comment from gundevel Dec 13, 2018
@ymcatwincities ymcatwincities deleted a comment from gundevel Dec 13, 2018
@ddrozdik
Copy link
Contributor

retest this please

@gundevel
Copy link
Collaborator

Build comment file:

Environment Link
Fresh OpenY installation - Rose theme http://openy.cibox.tools/build1187
Fresh OpenY installation - Lily theme http://lily.openy.cibox.tools/build1187
Upgraded(upgrade path) installation http://upgrade.openy.cibox.tools/build1187
Installation process http://install.openy.cibox.tools/build1187/install.php

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/1187/

@gundevel
Copy link
Collaborator

Acessibility Sniffer: front page checking WCAG2AA http://openy.cibox.tools/build1187/frontWCAG2AAhtmlcs.txt
Acessibility Sniffer: join page checking WCAG2AA http://openy.cibox.tools/build1187/joinWCAG2AAhtmlcs.txt
Acessibility Sniffer: locations page checking WCAG2AA http://openy.cibox.tools/build1187/locationsWCAG2AAhtmlcs.txt
Acessibility Sniffer: schedules page checking WCAG2AA http://openy.cibox.tools/build1187/schedulesWCAG2AAhtmlcs.txt
Acessibility Sniffer: blog page checking WCAG2AA http://openy.cibox.tools/build1187/blogWCAG2AAhtmlcs.txt

@ddrozdik ddrozdik added PR: Tested Manually tested. Green light for merging and removed PR: Needs Testing Manual testing is needed labels Dec 13, 2018
@ddrozdik ddrozdik merged commit 328bca5 into ymcatwincities:8.x-2.x Dec 13, 2018
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
PR: Code Reviewed Code reviewed. Ready for QA PR: Tested Manually tested. Green light for merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants