Skip to content

Commit e15a4a5

Browse files
committed
improved library layout
1 parent 744d467 commit e15a4a5

File tree

4 files changed

+22
-69
lines changed

4 files changed

+22
-69
lines changed

classes/Visualizer/Module/Admin.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -280,7 +280,7 @@ public function renderLibraryPage() {
280280
}
281281

282282
// Added by Ash/Upwork
283-
$filterByMeta = filter_input( INPUT_GET, 'filter', FILTER_SANITIZE_STRING );
283+
$filterByMeta = filter_input( INPUT_GET, 's', FILTER_SANITIZE_STRING );
284284
if ( $filterByMeta ) {
285285
$query = array(
286286
'key' => Visualizer_Plugin::CF_SETTINGS,

classes/Visualizer/Module/Chart.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,7 @@ public function getCharts() {
130130
) ),
131131
);
132132

133-
$filter = filter_input( INPUT_GET, 'filter', FILTER_SANITIZE_STRING );
133+
$filter = filter_input( INPUT_GET, 's', FILTER_SANITIZE_STRING );
134134
if ( $filter && in_array( $filter, Visualizer_Plugin::getChartTypes() ) ) {
135135
$query_args['meta_query'] = array(
136136
array(

classes/Visualizer/Render/Library.php

Lines changed: 16 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -87,36 +87,37 @@ private function _renderChartBox( $placeholder_id, $chart_id ) {
8787
private function _renderLibrary() {
8888
// Added by Ash/Upwork
8989
$filterBy = null;
90-
if ( isset( $_GET['filter'] ) && strlen( $_GET['filter'] ) > 0 ) {
91-
$filterBy = filter_input( INPUT_GET, 'filter', FILTER_SANITIZE_STRING );
90+
if ( isset( $_GET['s'] ) && strlen( $_GET['s'] ) > 0 ) {
91+
$filterBy = filter_input( INPUT_GET, 's', FILTER_SANITIZE_STRING );
9292
}
93-
echo '<div id="visualizer-search"><form action="" method="get">
94-
<input type="text" name="filter" value="' . $filterBy . '">
95-
<input type="hidden" name="page" value="visualizer">
96-
<input type="submit" class="button button-secondary" value="' . esc_attr__( 'Search', 'visualizer' ) . '">
97-
</form></div>';
93+
9894
// Added by Ash/Upwork
9995
echo '<div id="visualizer-types" class="visualizer-clearfix">';
100-
echo '<ul>';
96+
echo '<ul class="subsubsub">';
10197
foreach ( $this->types as $type => $array ) {
10298
$label = $array['name'];
103-
$link = '<a class="page-numbers" href="' . esc_url( add_query_arg( array( 'type' => $type, 'vpage' => false ) ) ) . '">';
99+
$link = '<a class=" " href="' . esc_url( add_query_arg( array( 'type' => $type, 'vpage' => false ) ) ) . '">';
104100
if ( ! $array['enabled'] ) {
105-
$link = "<a class='pro-upsell page-numbers' href='" . Visualizer_Plugin::PRO_TEASER_URL . "' target='_blank'>";
101+
$link = "<a class=' visualizer-pro-only' href='" . Visualizer_Plugin::PRO_TEASER_URL . "' target='_blank'>";
106102
}
107-
echo '<li class="visualizer-list-item">';
103+
echo '<li class="visualizer-list-item all">';
108104
if ( $type == $this->type ) {
109-
echo '<a class="page-numbers current" href="', esc_url( add_query_arg( 'vpage', false ) ), '">';
105+
echo '<a class=" current" href="', esc_url( add_query_arg( 'vpage', false ) ), '">';
110106
echo $label;
111107
echo '</a>';
112108
} else {
113109
echo $link;
114110
echo $label;
115111
echo '</a>';
116112
}
117-
echo '</li>';
113+
echo ' | </li>';
118114
}
119115
echo '</ul>';
116+
echo '<form action="" method="get"><p id="visualizer-search" class="search-box">
117+
<input type="search" name="s" value="' . $filterBy . '">
118+
<input type="hidden" name="page" value="visualizer">
119+
<input type="submit" id="search-submit" class="button button-secondary" value="' . esc_attr__( 'Search', 'visualizer' ) . '">
120+
</p> </form>';
120121
echo '</div>';
121122

122123
if ( ! empty( $this->charts ) ) {
@@ -127,9 +128,9 @@ private function _renderLibrary() {
127128
echo '</div>';
128129

129130
if ( is_array( $this->pagination ) ) {
130-
echo '<ul class="visualizer-library-pagination">';
131+
echo '<ul class=" subsubsub">';
131132
foreach ( $this->pagination as $page ) {
132-
echo '<li class="visualizer-list-item">', $page, '</li>';
133+
echo '<li class="all">', $page, '</li>';
133134
}
134135
echo '</ul>';
135136
}

css/library.css

Lines changed: 4 additions & 52 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,9 @@
2020
#visualizer-types {
2121
margin: 20px 0 10px;
2222
}
23-
23+
.visualizer-list-item .visualizer-pro-only{
24+
text-decoration: line-through;
25+
}
2426
#visualizer-library {
2527
margin: 20px 0;
2628
}
@@ -137,59 +139,9 @@
137139
margin-right: 20px;
138140
}
139141

140-
.page-numbers {
141-
padding: 5px 10px;
142-
border-radius: 2px;
143-
color: white !important;
144-
background-color: #aaa;
145-
-webkit-box-shadow: 0 0 5px #ccc;
146-
box-shadow: 0 0 5px #ccc;
147-
font-weight: bold;
148-
text-decoration: none;
149-
}
150-
151-
.page-numbers.current,
152-
a.page-numbers:hover {
153-
background-color: lightcoral;
154-
-webkit-box-shadow: 0 0 5px lightpink;
155-
box-shadow: 0 0 5px lightpink;
156-
}
157142

158-
.visualizer-library-pagination {
159-
display: none;
160-
}
161143

162144
#visualizer-search form {
163-
display: -webkit-box;
164-
display: -webkit-flex;
165-
display: -ms-flexbox;
166-
display: flex;
167-
168-
-webkit-align-items: center;
169-
align-items: center;
170-
-webkit-box-align: center;
171-
-ms-flex-align: center;
172-
}
173-
174-
#visualizer-search > form > input[type="text"]:nth-child(1) {
175-
height: 30px;
176-
}
177-
178-
#visualizer-search .button-secondary {
179-
margin-left: 20px;
180-
padding: 5px 10px;
181-
border-radius: 5px;
182-
color: white;
183-
background-color: lightcoral;
184-
-webkit-box-shadow: 0 0 5px lightpink;
185-
box-shadow: 0 0 5px lightpink;
186-
font-size: 14px;
187-
font-weight: bold;
188-
line-height: 17px;
189-
text-decoration: none;
145+
text-align: right;
190146
}
191147

192-
#visualizer-search .button-secondary:hover {
193-
border-color: lightpink;
194-
background-color: lightpink;
195-
}

0 commit comments

Comments
 (0)