search

EdisonWangB / pcmanx-gtk2

Automatically exported from code.google.com/p/pcmanx-gtk2
GNU General Public License v2.0
0 stars 0 forks source link

1.1版無論在哪個分頁按滑鼠中鍵都關到所在分頁 #67

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
What steps will reproduce the problem?
1.開啟不只一個分頁
2.在分頁欄任意處按滑鼠中鍵
3.出現確認關閉的對話框，按確定後關到正在看的分頁。

What is the expected output? What do you see instead?
像Chromium一樣在哪個分頁上按滑鼠中鍵就關掉哪一個。

What version of the product are you using? On what operating system?
PCManX1.1 on Lubuntu 12.10.

Please provide any additional information below.

Original issue reported on code.google.com by young811...@gmail.com on 29 Jan 2013 at 2:26

GoogleCodeExporter commented 8 years ago 
這看起來不算是bug，比較像是開新的功能；原本的source 
code是寫成關正在瀏覽中的分頁。

Original comment by taihsian...@gmail.com on 8 Sep 2013 at 6:12

GoogleCodeExporter commented 8 years ago 
實作得差不多了，再把code整理一下過兩天把patch送上

Original comment by taihsian...@gmail.com on 8 Sep 2013 at 5:28

GoogleCodeExporter commented 8 years ago 
patch做好了，請大家指教，謝謝！

Original comment by taihsian...@gmail.com on 13 Sep 2013 at 8:48

Attachments:

GoogleCodeExporter commented 8 years ago 
感謝提交貢獻！大致看了 patch，我覺得 
CMainFrame::CloseConByMidClick 
命名恐怕會讓人誤解，畢竟實際行為和 CMainFrame::CloseCon 
不同，而且一個 method 
名稱竟然和按鍵動作有關，這樣不直覺。

另外，請留意 coding style

Original comment by jserv.tw@gmail.com on 14 Sep 2013 at 5:08

GoogleCodeExporter commented 8 years ago 
感謝建議，根據建議修改了patch, 最新的patch如附件，
包括：
1. 
將CMainFrame::CloseConByMidClick命名更改為CMainFrame::CloseConAndPageSwitc
h
2. 統一method中if和for迴圈大括號的位置
   a. 原則上採用換行並對齊左跨號與右跨號(與src/mainframe.cpp中大部份的coding style統一)
   b. src/mainframe.cpp中另有少部份的method左括號不換行的方式撰寫，新增程式碼在這種method中的時候則繼承這樣的風格不另外做更動

----

編譯並執行過，測試方式為
1.打開三個tab,分別任意連上不同的站台
2.左鍵點選中間的tab, 
用中鍵關閉最左邊的tab，驗收是否確實關閉最左邊的tab，並��
�關閉後連線畫面跳回原本中間的tab
3.再開一個tab，連上另外一個站台，使總tab數目保持在三個
4.左鍵點選中間的tab, 
用中鍵關閉最右邊的tab，驗收是否確實關閉最右邊的tab，並��
�關閉後連線畫面跳回原本中間的tab
5.再開一個tab，連上另外一個站台，使總tab數目保持在三個
6.左鍵點選中間的tab, 
用中鍵關閉中間的tab，驗收是否確實關閉中間的tab
7.再開一個tab，連上另外一個站台，使總tab數目保持在三個
8.左鍵點選最左邊的tab, 
用中鍵關閉最右邊的tab，驗收是否確實關閉最右邊的tab，並��
�關閉後連線畫面跳回原本的tab
9.再開一個tab，連上另外一個站台，使總tab數目保持在三個
10.左鍵點選最右邊的tab, 
用中鍵關閉最左邊的tab，驗收是否確實關閉最左邊的tab，並��
�關閉後連線畫面跳回原本的tab

---

請大家不吝指教，謝謝！

Original comment by taihsian...@gmail.com on 23 Oct 2013 at 5:53

Attachments:

GoogleCodeExporter commented 8 years ago 
Some suggestion

1 Please add "Signed-of-by:" and re-send it again.
  用 git format-patch -s 

2.把  OnCloseConByMidClick 換成  OnCloseSelectCon 
  因為這個function 跟中鍵一點關係都沒，中鍵是外部的行為所掌控的 

3.中間你會pop dialog 問是不是真的要關掉，那部份跟 OnCloseCon 
裡做的事一模一樣，另外抽出來一個function 
吧。(這樣又多一個patch 了賺)

不過測過是沒問題的

Original comment by cmc...@gmail.com on 7 Nov 2013 at 2:45

GoogleCodeExporter commented 8 years ago 

謝謝大家的指教，新的patch如附件，說明如下：
  1. 0001-Close-tabs-selected-by-middle-clicks.patch
    * 基本上和comment 5是一樣的東西，但根據comment 6的建議將OnCloseConByMidClick重新命名為OnCloseSelectCon
  2. 0002-Modulize-query-dialogue-on-closing-connection.patch
    * 也是根據comment 6的建議把詢問視窗的function抽出來
      * 重構的時候考慮往前相容，因此保留OnCloseCon的interface，只改寫他其中詢問視窗的實作
      * 除了comment 5的測試外，另外有測試ctrl+del, alt+w, 還有滑鼠點擊x是否可以正常回答「Close Connection」的詢問（包括cancel/ok的回答都有測試），避免重構的時候把原來的功能改爛掉了

Original comment by taihsian...@gmail.com on 24 Nov 2013 at 6:19

Attachments:

GoogleCodeExporter commented 8 years ago 

hi, 請問這個功能團隊是否有打算加進去，
或是patch的寫法上有什麼不妥、可以再改進的地方，
希望大家可以不吝指教、討論 :)
(patch擺得有點久了 XDD)

Original comment by taihsian...@gmail.com on 24 Dec 2013 at 3:57

GoogleCodeExporter commented 8 years ago 
Merged.  Thanks!
git commit 628ccbfca9352275182bea7ca2db4e329ea7d186 and 
576e54a46374194320b12350ef9c0f4336d9e1f1

Original comment by jserv.tw@gmail.com on 24 Dec 2013 at 4:10

GoogleCodeExporter commented 8 years ago

Original comment by jserv.tw@gmail.com on 24 Dec 2013 at 4:10