Skip to content

Add search and dropdown menu with book categories - #58

Open
YuriiHaidamaka wants to merge 160 commits into
masterfrom
add_search_and_dropdown
Open

Add search and dropdown menu with book categories#58
YuriiHaidamaka wants to merge 160 commits into
masterfrom
add_search_and_dropdown

Conversation

@YuriiHaidamaka

@YuriiHaidamakaYuriiHaidamaka commented Apr 16, 2018

Copy link
Copy Markdown
Contributor

In this PR we have created <exlibris-category-dropdown> and <exlibris-book-search>web components according to next user tasks:

-Add category drop-down selector for All Books item
-Add search input to the <app-header>

The <exlibris-category-dropdown> shows a dropdown menu with categories.
The <exlibris-category-dropdown> works on the following URL : /#/all-books/category/{categoryName}
Each item from dropdown menu is a link of the next format: /#/all-books/category/{categoryName}. This link will show all books from the library from this category. After clicking on the All Books menu item the filtering is canceled and <exlibris-category-dropdown> is reseted.

The <exlibris-category-dropdown> should only be shown for All Books. In the web component exlibris-book-grid was added slot for this purpose.

The <exlibris-book-search> allows to search allBooks array.
The <exlibris-book-search> works on the following URL : /?q={query}#/all-books .
After 500 milliseconds as the query was changed the exlibris-book-search-query-changed will be fired and the corresponding action will be dispatch to the redux store.

Page with all books has the next design now :

image

With open <exlibris-category-dropdown> :

image

Result of emty search:

image

The result of the polymer lint:
image

YegorUdovchenkoand others added 30 commits March 26, 2018 14:22
# Conflicts:
#	client-web/src/exlibris-app/exlibris-app.html
# Conflicts:
#	client-web/src/exlibris-app/exlibris-app.html
# Conflicts:
#	client-web/src/exlibris-app/exlibris-app.html
#	client-web/src/exlibris-app/shell/redux-store.html
# Conflicts:
#	client-web/src/exlibris-app/exlibris-app.html
# Conflicts:
#	client-web/bower.json
#	client-web/src/exlibris-app/exlibris-app.html
@YuriiHaidamakaYuriiHaidamaka added the wip Work in progress label Apr 16, 2018
@YuriiHaidamakaYuriiHaidamaka changed the title Add search and dropdownAdd search and dropdown menu with book categoriesApr 16, 2018
@codecov

codecovBot commented Apr 16, 2018

Copy link
Copy Markdown

Codecov Report

Merging #58 into master will increase coverage by 0.83%.
The diff coverage is n/a.

@@ Coverage Diff @@## master #58 +/- ##
==========================================
+ Coverage 96.16% 96.99% +0.83% 
==========================================
Files 11 39 +28 Lines 860 1966 +1106 Branches 41 56 +15 ==========================================
+ Hits 827 1907 +1080 - Misses 23 42 +19 - Partials 10 17 +7

@YuriiHaidamakaYuriiHaidamaka self-assigned this Apr 16, 2018
@YuriiHaidamaka
YuriiHaidamakaforce-pushed the add_search_and_dropdown branch from 43d09d6 to 6cf875aCompareApril 16, 2018 16:00
# Conflicts:
#	client-web/src/exlibris-app/book/book-values.html
#	client-web/src/exlibris-app/book/exlibris-book-grid.html
#	client-web/src/exlibris-app/book/exlibris-book-tile.html
#	client-web/src/exlibris-app/exlibris-app.html
#	client-web/src/exlibris-app/reducers/exlibris-reducer.html
#	client-web/test/exlibris-app/exlibris-app_store_test.html
@YuriiHaidamakaYuriiHaidamaka removed the wip Work in progress label Apr 17, 2018
YegorUdovchenkoand others added 4 commits April 18, 2018 11:24
# Conflicts:
#	client-web/src/exlibris-app/actions/exlibris-actions.html
#	client-web/src/exlibris-app/book/book-values.html
#	client-web/src/exlibris-app/book/exlibris-book-grid.html
#	client-web/src/exlibris-app/book/exlibris-book-tile.html
#	client-web/src/exlibris-app/exlibris-app.html
#	client-web/src/exlibris-app/reducers/exlibris-reducer.html
# Conflicts:
#	client-web/src/exlibris-app/book/book-values.html
#	client-web/src/exlibris-app/book/exlibris-book-grid.html
#	client-web/src/exlibris-app/book/exlibris-book-tile.html
#	client-web/src/exlibris-app/exlibris-app.html
#	client-web/src/exlibris-app/reducers/exlibris-reducer.html
#	client-web/test/exlibris-app/exlibris-app_store_test.html
@YuriiHaidamaka
YuriiHaidamakaforce-pushed the add_search_and_dropdown branch from 62de68b to fab8e4eCompareMay 3, 2018 13:48
YegorUdovchenkoand others added 8 commits May 4, 2018 11:05
# Conflicts:
#	client-web/src/exlibris-app/actions/exlibris-action.html
#	client-web/src/exlibris-app/actions/exlibris-actions.html
#	client-web/src/exlibris-app/book/book-values.html
#	client-web/src/exlibris-app/book/exlibris-book-grid.html
#	client-web/src/exlibris-app/book/exlibris-book-tile.html
#	client-web/src/exlibris-app/exlibris-app.html
#	client-web/src/exlibris-app/reducers/exlibris-reducer.html
#	client-web/test/exlibris-app/exlibris-app_store_test.html
Sign up for freeto 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.

2 participants

@YuriiHaidamaka@YegorUdovchenko