Skip to content

Commit e4085f5

Browse files
committed
[refactor] improve search results
1 parent 4e82d79 commit e4085f5

File tree

6 files changed

+60
-68
lines changed

6 files changed

+60
-68
lines changed

‎css/link.css‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -70,8 +70,8 @@
7070
.link-item-placeholder{
7171
background-color:rgba(var(--theme-accent),0.2);
7272
border-radius:var(--theme-radius);
73-
width:100%;
74-
height:100%;
73+
width:100%;
74+
height:100%;
7575
position: relative;
7676
top:50%;
7777
left:50%;
@@ -398,7 +398,7 @@
398398

399399
.link-empty-heading{
400400
color:rgb(var(--gray-16));
401-
font-size:1.5rem;
401+
margin-bottom:0.5em;
402402
}
403403

404404
.is-link-style-list .link-name{

‎js/control.js‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1279,53 +1279,53 @@ var control = (function(){
12791279
type: "radio",
12801280
func: function(){
12811281
dependents();
1282-
search.render();
1282+
search.render.engine();
12831283
}
12841284
},{
12851285
element: helper.e(".control-header-search-engine-duckduckgo"),
12861286
path: "header.search.engine.selected",
12871287
type: "radio",
12881288
func: function(){
12891289
dependents();
1290-
search.render();
1290+
search.render.engine();
12911291
}
12921292
},{
12931293
element: helper.e(".control-header-search-engine-youtube"),
12941294
path: "header.search.engine.selected",
12951295
type: "radio",
12961296
func: function(){
12971297
dependents();
1298-
search.render();
1298+
search.render.engine();
12991299
}
13001300
},{
13011301
element: helper.e(".control-header-search-engine-giphy"),
13021302
path: "header.search.engine.selected",
13031303
type: "radio",
13041304
func: function(){
13051305
dependents();
1306-
search.render();
1306+
search.render.engine();
13071307
}
13081308
},{
13091309
element: helper.e(".control-header-search-engine-custom"),
13101310
path: "header.search.engine.selected",
13111311
type: "radio",
13121312
func: function(){
13131313
dependents();
1314-
search.render();
1314+
search.render.engine();
13151315
}
13161316
},{
13171317
element: helper.e(".control-header-search-engine-custom-name"),
13181318
path: "header.search.engine.custom.name",
13191319
type: "text",
13201320
func: function(){
1321-
search.render();
1321+
search.render.engine();
13221322
}
13231323
},{
13241324
element: helper.e(".control-header-search-engine-custom-url"),
13251325
path: "header.search.engine.custom.url",
13261326
type: "text",
13271327
func: function(){
1328-
search.render();
1328+
search.render.engine();
13291329
}
13301330
},{
13311331
element: helper.e(".control-header-search-text-align-left"),
@@ -1752,7 +1752,7 @@ var control = (function(){
17521752
func: function(){
17531753
render();
17541754
dependents();
1755-
search.render();
1755+
search.render.engine();
17561756
}
17571757
},{
17581758
element: helper.e(".control-link-item-display-show"),

‎js/link.js‎

Lines changed: 8 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -531,43 +531,21 @@ var link = (function(){
531531

532532
var_makeEmptySearch=function(){
533533
varsearchInput=helper.e(".search-input");
534-
vardiv=helper.makeNode({
535-
tag: "div",
536-
attr: [{
537-
key: "class",
538-
value: "link-empty"
539-
}]
540-
});
541-
varh1=helper.makeNode({
542-
tag: "h1",
543-
attr: [{
544-
key: "class",
545-
value: "link-empty-heading"
546-
}],
547-
text: "No matching bookmarks found"
548-
});
534+
vardiv=helper.node("div|class:link-empty");
535+
varh1=helper.node("h1:No matching bookmarks found|class:link-empty-heading");
536+
varpara=helper.node("p:Enter to Search "+state.get().header.search.engine[state.get().header.search.engine.selected].name+"|class:small muted");
549537
div.appendChild(h1);
538+
div.appendChild(para);
550539
returndiv;
551540
};
552541

553542
var_makeEmptyBookmarks=function(){
554543
varsearchInput=helper.e(".search-input");
555-
vardiv=helper.makeNode({
556-
tag: "div",
557-
attr: [{
558-
key: "class",
559-
value: "link-empty"
560-
}]
561-
});
562-
varh1=helper.makeNode({
563-
tag: "h1",
564-
attr: [{
565-
key: "class",
566-
value: "link-empty-heading"
567-
}],
568-
text: "No bookmarks added"
569-
});
544+
vardiv=helper.node("div|class:link-empty");
545+
varh1=helper.node("h1:No bookmarks added|class:link-empty-heading");
546+
varpara=helper.node("p:Why not add some?|class:small muted");
570547
div.appendChild(h1);
548+
div.appendChild(para);
571549
returndiv;
572550
};
573551

‎js/search.js‎

Lines changed: 39 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -4,22 +4,26 @@ var search = (function(){
44
varsearchInput=helper.e(".search-input");
55
varsearchClear=helper.e(".search-clear");
66
searchInput.addEventListener("input",function(){
7-
_toggle(this);
8-
_searchClear();
7+
_toggle();
8+
render.clear.button();
99
link.clear();
1010
link.render.item.all();
1111
sortable(".link-area");
1212
},false);
1313
searchClear.addEventListener("click",function(){
14-
_toggle(this);
15-
_searchClear();
16-
clear();
14+
render.clear.input();
15+
_toggle();
16+
render.clear.button();
17+
link.clear();
18+
link.render.item.all();
19+
sortable(".link-area");
1720
},false);
1821
};
1922

20-
var_toggle=function(input){
23+
var_toggle=function(){
2124
varhtml=helper.e("html");
22-
if(input.value!=""){
25+
varsearchInput=helper.e(".search-input");
26+
if(searchInput.value!=""){
2327
helper.setObject({
2428
object: state.get(),
2529
path: "search",
@@ -36,16 +40,6 @@ var search = (function(){
3640
};
3741
};
3842

39-
var_searchClear=function(){
40-
varsearchInput=helper.e(".search-input");
41-
varsearchClear=helper.e(".search-clear");
42-
if(state.get().search){
43-
searchClear.removeAttribute("disabled");
44-
}else{
45-
searchClear.setAttribute("disabled","");
46-
};
47-
};
48-
4943
varget=function(){
5044
varsearchInput=helper.e(".search-input");
5145
if(state.get().search){
@@ -71,7 +65,21 @@ var search = (function(){
7165
};
7266
};
7367

74-
varrender=function(){
68+
varrender={
69+
engine: function(){
70+
_renderEngine();
71+
},
72+
clear: {
73+
input: function(){
74+
_renderClearInput();
75+
},
76+
button: function(){
77+
_renderClearButton();
78+
}
79+
}
80+
};
81+
82+
var_renderEngine=function(){
7583
varsearch=helper.e(".search");
7684
varsearchInput=helper.e(".search-input");
7785
varplaceholder="";
@@ -85,13 +93,19 @@ var search = (function(){
8593
search.setAttribute("action",state.get().header.search.engine[state.get().header.search.engine.selected].url);
8694
};
8795

88-
varclear=function(){
96+
var_renderClearButton=function(){
97+
varsearchClear=helper.e(".search-clear");
98+
if(state.get().search){
99+
searchClear.removeAttribute("disabled");
100+
}else{
101+
searchClear.setAttribute("disabled","");
102+
};
103+
};
104+
105+
var_renderClearInput=function(){
89106
varsearchInput=helper.e(".search-input");
90107
searchInput.value="";
91108
searchInput.focus();
92-
link.clear();
93-
link.render.item.all();
94-
sortable(".link-area");
95109
};
96110

97111
var_focus=function(){
@@ -104,16 +118,16 @@ var search = (function(){
104118

105119
varinit=function(){
106120
bind();
107-
render();
121+
render.engine();
122+
_toggle();
108123
_focus();
109124
};
110125

111126
// exposed methods
112127
return{
113128
init: init,
114129
get: get,
115-
render: render,
116-
clear: clear
130+
render: render
117131
};
118132

119133
})();

‎js/version.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
varversion=(function(){
22

3-
varcurrent="3.16.3";
3+
varcurrent="3.17.0";
44

55
varcompare=function(a,b){
66
varpa=a.split(".");

‎manifest.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
"name": "nightTab",
33
"short_name": "nightTab",
44
"description": "A neutral new tab page accented with a chosen colour. Customise the layout, style, background and bookmarks in nightTab.",
5-
"version": "3.16.3",
5+
"version": "3.17.0",
66
"manifest_version": 2,
77
"chrome_url_overrides":{
88
"newtab": "index.html"

0 commit comments

Comments
(0)