Skip to content
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

Adjust search function in child theme #4

Open
SRD75 opened this issue Jul 27, 2018 · 3 comments
Open

Adjust search function in child theme #4

SRD75 opened this issue Jul 27, 2018 · 3 comments

Comments

@SRD75
Copy link

SRD75 commented Jul 27, 2018

Hi,

We'd like to implement the search functionality found @ https://www.insightdesign.com.au/ onto https://www-new.insightdesign.com.au/

That is, when we click the search icon, we'd like .search-bg to become active, so that the view darkens.

How do I make .search-bg open when clicking the search icon, instead of the default search form?

Thanks,
Steve

Make sure these boxes are checked before submitting your issue.

  • [y] You are using this for WordPress.

  • [y] You have Divi installed.

  • [y] You have lint your code to ensure the problem isn't caused by your code.

  • [y] You have already searched the GitHub issues and it has not been reported before.

@etstaging
Copy link

QA Checklist

Reporter

  • Helpful title used  
    • Helpful title format: Scope(s) :: One Sentence To Describe The Issue
    • Example: VB :: Floating Toolbar :: Jumps To Top Of Page
  • Detailed description
    • Description should usually be 150+ characters.
  • Steps to reproduce included
  • Support Ticket URL included
  • Image and/or gif

Developer

  • Steps to reproduce proven solved
  • Test included and coverage acceptable and all Tests passing?
  • PR description has layman's description of problem and solution and "Areas Affected" listed/explained?
  • PR branch includes latest commits from master branch (at the time of checking)?
  • Estimated points on issue
  • Proper code documentation and conventions?
  • Proper RTL appearance
  • i18n used if applicable
  • Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
  • Database saved/updated? Data sanitized and validated?
  • Variable output properly escaped and validated?

Peer Review

  • PR branch includes latest commits from master branch (at the time of checking)?
  • Steps to reproduce proven solved
  • Affected areas are tested and ok
  • Proper code documentation and conventions
  • Proper RTL appearance
  • i18n used if applicable
  • Test(s) included and coverage acceptable
  • Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
  • Database saved/updated? Data sanitized and validated?
  • Variable output properly escaped and validated?

Quality Assurance

  • All tests passed on CircleCI.
  • Steps to reproduce passed, the fix works.
  • No errors in console or output
  • Affected areas are tested and ok
  • The fix does not change a customer's website in any surprising or unintended way
    • If Divi's design is adjusted, settings are migrated gracefully and invisibly

Design Review

  • UI is to spec
  • UX is to spec
  • Design is to spec
  • No surprising visual changes to a customer's website will be experienced after updating
    • Any/all visual changes are migrated gracefully

Dev Review

  • Test(s) included and coverage acceptable
  • Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
  • Database saved/updated? Data sanitized and validated?
  • Variable output properly escaped and validated?
  • i18n used if applicable
  • Implementation acceptable, DRY, Scaleable, future safe
  • Best practices used, cleverness and grokage, ES6 use, etc.

@SRD75
Copy link
Author

SRD75 commented Jul 31, 2018

I've adjusted my initial post.

@SRD75
Copy link
Author

SRD75 commented Aug 2, 2018

Can you delete or close this request, as we have resolved the issue? Thanks.

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

No branches or pull requests

2 participants