sakura icon indicating copy to clipboard operation
sakura copied to clipboard

空のEOF行に行番号が表示されない

Open beru opened this issue 6 years ago • 6 comments

他のソフト、例えば Visual Studio, Visual Studio Code, Mery, GVIM, 秀丸 だと改行コードがまだ無くても行番号が表示されますがサクラエディタの場合は表示されません。

デフォルトの動作を変更してほしいわけでは無いのですが設定で切り替えられると良いなと思います。

beru avatar Dec 01 '18 21:12 beru

スペースがあると行番号が表示されますね。

Ver2.3.2.0

image

arigayas avatar Dec 01 '18 22:12 arigayas

スペースがあると行番号が表示されますね。

あ、本当ですね…。確認が不十分でした。という事でタイトルを変更しました。

beru avatar Dec 01 '18 22:12 beru

実装で言うとこの辺ですな。

https://github.com/sakura-editor/sakura/blob/a950d23b435a87b3a2cbe79048ed2d48e01152b1/sakura_core/view/CTextDrawer.cpp#L453-L459

たぶんですが、ここ削ったらうまく動くんじゃね?という雰囲気です。 これは、まったく検証しとらん無責任発言なので鵜呑みにしないでくださいね:smile:

berryzplus avatar Dec 02 '18 08:12 berryzplus

ファイルの行数にカウントされない終端空行には行番号を表示しない、というのも理屈としては分からなくもないです。しかし他のエディタが軒並み表示してるのはその方がユーザの感覚としては違和感が無いということなんですかね?入力したらその行が有効になるわけで未来の目安にはなるので。

beru avatar Dec 02 '18 09:12 beru

どうなんでしょう。

上記実装の存在理由は、行情報が未生成の場合におかしなことにならないための対策と考えられます。でも、elseパートの行情報取得のところを見ると「行情報がないケース」への対策が入っています。

https://github.com/sakura-editor/sakura/blob/a950d23b435a87b3a2cbe79048ed2d48e01152b1/sakura_core/view/CTextDrawer.cpp#L488-L492

削ったらうまいこと動くんじゃね?の根拠はこのへんでした。 (対策入ってるから大丈夫じゃないかなぁ、という期待w)

berryzplus avatar Dec 02 '18 09:12 berryzplus

ああ、ダメかw うまいことは動かなそう(" "になる)

berryzplus avatar Dec 02 '18 09:12 berryzplus