Skip to content

Search topbar#2765

Merged
josepjaume merged 1 commit intomasterfrom
2714-top-header-search-box
Feb 21, 2018
Merged

Search topbar#2765
josepjaume merged 1 commit intomasterfrom
2714-top-header-search-box

Conversation

@Crashillo
Copy link
Copy Markdown
Contributor

@Crashillo Crashillo commented Feb 19, 2018

🎩 What? Why?

Design of the top header search box.
IMPORTANT It does not contain the home page search.

📌 Related Issues

📷 Screenshots (optional)

Desktop:
imagen

Mobile:
imagen

@ghost ghost assigned Crashillo Feb 19, 2018
@ghost ghost added the in-progress label Feb 19, 2018
@Crashillo Crashillo requested a review from josepjaume February 19, 2018 17:04
@Crashillo Crashillo mentioned this pull request Feb 19, 2018
2 tasks
This was referenced Feb 19, 2018
@agustibr
Copy link
Copy Markdown
Contributor

agustibr commented Feb 21, 2018

@Crashillo this PR has a failing test

@decidim/lot-core we need this PR accepted to continue with the Search engine MVP · Issue #2707 · decidim/decidim

@Crashillo
Copy link
Copy Markdown
Contributor Author

You mean ci/circleci:core?? Better ask @decidim/lot-core, I'm afraid I cannot help there

@mrcasals
Copy link
Copy Markdown
Contributor

I'm rerunning the job, see if it is solved.

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 21, 2018

Codecov Report

Merging #2765 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2765      +/-   ##
==========================================
+ Coverage   98.85%   98.86%   +<.01%     
==========================================
  Files        1534     1534              
  Lines       36091    36091              
==========================================
+ Hits        35679    35680       +1     
+ Misses        412      411       -1

@josepjaume josepjaume merged commit dae9914 into master Feb 21, 2018
@josepjaume josepjaume deleted the 2714-top-header-search-box branch February 21, 2018 14:10
@ghost ghost removed the status: ready label Feb 21, 2018
@agustibr
Copy link
Copy Markdown
Contributor

🙌

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

Successfully merging this pull request may close these issues.

5 participants