Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

複数タブに対応 #45

Merged
merged 25 commits into from
Jul 30, 2020
Merged

複数タブに対応 #45

merged 25 commits into from
Jul 30, 2020

Conversation

Sota-Watanabe
Copy link
Member

@Sota-Watanabe Sota-Watanabe commented Jul 29, 2020

タブを複数を作成, 削除できるようにしました.
細かいUIはまだ作成していません.

説明するにあたって以下の要素を持つものを1セットとしてdocumentと表現します.
https://github.com/cdlab-sit/editor/blob/4e61383be4610fdb7ac79341a39a1e6dc13f7950/src/renderer/reducks/editor/reducers.js#L13-L18

仕様

  • 対象となる部分(仮)をダブルクリックすると新規documentが作成されます.
  • 作成されたdocumentは独立しており, タブを切り替えたとしても範囲選択は保持されます.
  • タブにあるFileStatusも独立しています.
  • タブを選択すると対象となるdocumentがアクティブになります.
  • FooterのCharCountはアクティブなdocumentについて表示されます.
  • タブにあるFileStatusをクリックすると対象となるdocumentが削除されます.
  • 削除後は, その右隣にあるタブに対応したdocumentがアクティブになります. 削除するタブが右端だった場合, 更新後の右端のタブに対応したdocumentがアクティブになります.

未対応

  • ×押しても警告なしでタブが削除されます
  • タブを増やすとウィンドウが横に伸びます(タブの横幅を動的に変更する機能なし)
  • サイドバーのFileBlockは未実装

@Sota-Watanabe
Copy link
Member Author

Sota-Watanabe commented Jul 29, 2020

#42
Tabs->Tabに変更しました.

Copy link
Member

@higurashi-takuto higurashi-takuto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

とりあえず名前についてのコメントです。
動作に関してはもう一回別でコメントします。

src/renderer/components/Tab.jsx Outdated Show resolved Hide resolved
src/renderer/components/Main.jsx Outdated Show resolved Hide resolved
src/renderer/reducks/editor/actions.js Outdated Show resolved Hide resolved
src/renderer/reducks/editor/actions.js Outdated Show resolved Hide resolved
@higurashi-takuto
Copy link
Member

現時点で、動作周りは問題なさそうです。
未実装な部分は issue 立てるか、このまま、このプルリク内で作成するかするようにしましょう。

@Sota-Watanabe
Copy link
Member Author

#49 #48 #47
未実装な部分のissueを立てました.

また指摘部分の修正を行いました.
確認お願いします.

@higurashi-takuto higurashi-takuto merged commit 20d1c92 into master Jul 30, 2020
@higurashi-takuto higurashi-takuto deleted the multi-tab-no-module branch July 31, 2020 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants