- Git 12©2ch.net
457 ：デフォルトの名無しさん[sage]：2015/05/16(土) 01:25:17.31 ID:+UC07Y5W - >>455
githubは知らないけど、gitlab（最新版）なら
ちゃんと残る。
正確にはコメントやレビューはマージリクエスト（githubのプルリク相当）に書かないとダメ。
ブランチ（コミット）に対してのコメントであれば見れなくなる。（今は残ってる気もするけど）
|
- + JavaScript の質問用スレッド vol.118 + [転載禁止]©2ch.net
59 ：デフォルトの名無しさん[sage]：2015/05/16(土) 01:39:13.60 ID:+UC07Y5W - >>57はあまり良くないので訂正する。
その前に、>>55のコードは無駄があるので先に綺麗にしておく。
$(function() {
$(window).scroll( function() {
var enabled = $(this).scrollTop() > $("#fixedNavi").top;
navBox.toggleClass("fixBox", enabled);
}
});
まず見た目の話なので、クラスを使う。
ボタンが押された時に、bodyにクラスをつけたり外したりする。
$('#btn').click(function() {
$('body').toggleClass('scroll-bar-fixed');
});
fixBoxクラスの有無でCSSでスクロールバーの固定を行っていると思うが、
あとはCSSをちょこっと書き換えて、
fixBox { 省略 } // スクロールバーの固定解除モード
scroll-bar-fixed fixBox { 省略 } // スクロールバーの固定モード
のような感じで制御できる。
重要なのは、JavaScriptではクラスを使った"モード"の変移のみを行い、
それによって変わる見た目は、CSSを使う言うこと。
これをしないとJavaScriptのコードがすぐに複雑になってしまう。
|
- + JavaScript の質問用スレッド vol.118 + [転載禁止]©2ch.net
60 ：デフォルトの名無しさん[sage]：2015/05/16(土) 01:41:23.31 ID:+UC07Y5W - あ、navBox変数が無いｗ
まあ適当に修正してくれ。
|
- + JavaScript の質問用スレッド vol.118 + [転載禁止]©2ch.net
61 ：デフォルトの名無しさん[sage]：2015/05/16(土) 01:43:39.73 ID:+UC07Y5W - もう一箇所ミスってたんで、書きなおした。
$(function() {
$(window).scroll( function() {
var navBox = $("#fixedNavi");
var enabled = $(this).scrollTop() > navBox.offset().top;
navBox.toggleClass("fixBox", enabled);
}
});
|
- 今までみた絶望的なソースコード [転載禁止]©2ch.net
72 ：デフォルトの名無しさん[sage]：2015/05/16(土) 01:51:37.25 ID:+UC07Y5W - >>70
ロジックが同じかどうかでまとめるのではなくて、
役割で考えないとダメだよ。
同じロジックであってもまとめるべきではないこともたまにある。
例えばTwitter APIとAWS APIの内部で同じ処理をやっていたとしても、
役割はそれぞれ別なので分けるべきだろう。
もちろん汎用モジュールに分けられるようなものであれば
分けた方がいい。
|
- 【JavaScript】ｽｸﾘﾌﾟﾄ ﾊﾞﾄﾙﾛﾜｲﾔﾙ49【php,py,pl,rb】 [転載禁止]©2ch.net
485 ：デフォルトの名無しさん[sage]：2015/05/16(土) 12:49:49.59 ID:+UC07Y5W - jser.infoの宣伝するのやめろｗ
|
- + JavaScript の質問用スレッド vol.118 + [転載禁止]©2ch.net
63 ：デフォルトの名無しさん[sage]：2015/05/16(土) 13:07:55.59 ID:+UC07Y5W - 動くかどうかは知らん
$(function() {
var navBox = $("#fixedNavi");
var navTop = navBox.offset().top;
$(window).scroll(function() {
var enabled = $(this).scrollTop() > navTop;
navBox.toggleClass("fixBox", enabled);
});
$('#btn').click(function() {
$('body').toggleClass('scroll-bar-fixed');
});
});
.scroll-bar-fixed .fixBox {
position: fixed;
top: 0px;
}
<div id="fixedNavi">
〜サイドメニュー〜
</div>
<button id="btn">ボタン</button>
|