質問編集履歴
1
test
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
コードの評価願い
|
|
1
|
+
コードの評価願いコードの評価願い
|
test
CHANGED
|
@@ -1,73 +1,3 @@
|
|
|
1
1
|
こんばんは、お世話になります。
|
|
2
2
|
|
|
3
|
-
|
|
4
|
-
|
|
5
|
-
Wordpressにおいて、サイドバーに設置したul & liのナビゲーションメニューを、select & option にjQueryで書き換えました。コードの良し悪しを評価願います。
|
|
6
|
-
|
|
7
|
-
|
|
8
|
-
|
|
9
|
-
条件：
|
|
10
|
-
|
|
11
|
-
テーマファイルは触れない。
|
|
12
|
-
|
|
13
|
-
ulとliにクラスやIDは設定されていない。
|
|
14
|
-
|
|
15
|
-
|
|
16
|
-
|
|
17
|
-
```html
|
|
18
|
-
|
|
19
|
-
jQuery(document).ready(function( $ ){
|
|
20
|
-
|
|
21
|
-
|
|
22
|
-
|
|
23
|
-
let $ul = $( '.sidebar ul' );
|
|
24
|
-
|
|
25
|
-
let $li = $ul.find('li');
|
|
26
|
-
|
|
27
|
-
let li_length = $li.length;
|
|
28
|
-
|
|
29
|
-
|
|
30
|
-
|
|
31
|
-
let $select = $('<select onChange="location.href=value;"></select>');
|
|
32
|
-
|
|
33
|
-
$select.append( '<option value="">-</option>' );
|
|
34
|
-
|
|
35
|
-
|
|
36
|
-
|
|
37
|
-
for ( let i = 0 ; i < li_length ; i++ ){
|
|
38
|
-
|
|
39
|
-
let $href = $li.eq(i).children('a').prop('href');
|
|
40
|
-
|
|
41
|
-
let $text = $li.eq(i).children('a').text();
|
|
42
|
-
|
|
43
|
-
$select.append( '<option value="' + $href + '">' + $text +'</option>' );
|
|
44
|
-
|
|
45
|
-
}
|
|
46
|
-
|
|
47
|
-
|
|
48
|
-
|
|
49
|
-
$ul.replaceWith( $select );
|
|
50
|
-
|
|
51
|
-
|
|
52
|
-
|
|
53
|
-
// console.log( $select.html() );
|
|
54
|
-
|
|
55
|
-
|
|
56
|
-
|
|
57
|
-
});
|
|
58
|
-
|
|
59
|
-
|
|
60
|
-
|
|
61
|
-
```
|
|
62
|
-
|
|
63
|
-
|
|
64
|
-
|
|
65
|
-
ナビゲーションメニューは、外観→メニューから作るメニュー。
|
|
66
|
-
|
|
67
|
-
サイドバーは、外観→ウィジェットで設定できるもの。
|
|
68
|
-
|
|
69
|
-
つまり、外観→ウィジェットに行き、ナビゲーションメニューをドラッグして設置しました。
|
|
70
|
-
|
|
71
|
-
|
|
72
|
-
|
|
73
3
|
もっとこういう書き方がある、こう書いた方が速くなる、など評価や修正を頂けると嬉しいです。よろしくお願いいたします。
|