search

sakura-editor / sakura

SAKURA Editor (Japanese text editor for MS Windows)
https://sakura-editor.github.io/
Other
1.25k stars 163 forks source link

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

Open beru opened 5 years ago

beru commented 5 years ago

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

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

arigayas commented 5 years ago

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

Ver2.3.2.0

image

beru commented 5 years ago

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

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

berryzplus commented 5 years ago

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

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

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

beru commented 5 years ago

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

berryzplus commented 5 years ago

どうなんでしょう。

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

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

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

berryzplus commented 5 years ago

ああ、ダメかｗ うまいことは動かなそう（" "になる）