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

Sending a pull request. #2

Open
wants to merge 76 commits into
base: master
Choose a base branch
from
Open

Conversation

mustah
Copy link

@mustah mustah commented Jun 3, 2012

Hi Bas.

I've made some changes to this amazing plugin:

  1. Bug fix, where in intellij idea version 11.1.1 and 11.1.2 made the selection move down 2 steps in stead of one when pressing CMD+E.
  2. Some code clean ups.
  3. Created possibility to display a list of modified files in the default change list of the selected VCS. (I've personally mapped it to CTRL+E)

Please review the code and come back to me :-)

Thx and kind regards,
/Must

@BasLeijdekkers
Copy link
Owner

Hi Must,

Thanks for the pull request. I noticed you have managed to update the
plugin in the repository too. Sorry for the late reply by the way, but
I guess you're managing things without my help:-)

Shall I add you to the committers of my TabSwitch repository?

Thanks,
Bas

On Sun, Jun 3, 2012 at 10:24 PM, Must
[email protected]
wrote:

Hi Bas.

I've made some changes to this amazing plugin:

  1. Bug fix, where in intellij idea version 11.1.1 and 11.1.2 made the selection move down 2 steps in stead of one when pressing CMD+E.
  2. Some code clean ups.
  3. Created possibility to display a list of modified files in the default change list of the selected VCS.

Please review the code and come back to me :-)

Thx and kind regards,
/Must

You can merge this Pull Request by running:

 git pull https://github.com/mustah/TabSwitch master

Or you can view, comment on it, or merge it online at:

 #2

-- Commit Summary --

  • code reformatting
  • Added override annotation
  • TabSwitch version 3. Includes an additional popup chooser window, where the changes from default change list of your VCS is listed.
  • Added intellij idea project files too.
  • TabSwitch.iws is not needed in the repository.

-- File Changes --

M META-INF/plugin.xml (41)
A TabSwitch.iml (15)
A TabSwitch.ipr (321)
M build.xml (33)
A src/org/intellij/ideaplugins/tabswitch/ComponentFactory.java (58)
D src/org/intellij/ideaplugins/tabswitch/Handler.java (291)
A src/org/intellij/ideaplugins/tabswitch/ListCellRendererFactory.java (66)
A src/org/intellij/ideaplugins/tabswitch/ListSelectionListenerFactory.java (60)
D src/org/intellij/ideaplugins/tabswitch/PreviousTabAction.java (24)
D src/org/intellij/ideaplugins/tabswitch/SwitchTabAction.java (58)
D src/org/intellij/ideaplugins/tabswitch/TabAction.java (47)
A src/org/intellij/ideaplugins/tabswitch/TabSwitchProjectComponent.java (213)
M src/org/intellij/ideaplugins/tabswitch/TabSwitchSettings.java (69)
R src/org/intellij/ideaplugins/tabswitch/action/NextTabAction.java (23)
A src/org/intellij/ideaplugins/tabswitch/action/NextVcsChangeAction.java (28)
A src/org/intellij/ideaplugins/tabswitch/action/SwitchTabAction.java (57)
A src/org/intellij/ideaplugins/tabswitch/action/TabAction.java (60)
A src/org/intellij/ideaplugins/tabswitch/filefetchers/FileFetcher.java (23)
A src/org/intellij/ideaplugins/tabswitch/filefetchers/FileFetcherChangedFilesInVcs.java (66)
A src/org/intellij/ideaplugins/tabswitch/filefetchers/FileFetcherOpenTabFiles.java (34)

-- Patch Links --

 https://github.com/BasLeijdekkers/TabSwitch/pull/2.patch
 https://github.com/BasLeijdekkers/TabSwitch/pull/2.diff


Reply to this email directly or view it on GitHub:
#2

Bas Leijdekkers | Desert Island BV | T +31 6 2202 8877 | E
[email protected]

@mustah
Copy link
Author

mustah commented Jun 16, 2012

Hi Bas.

Yes, please do.

Thx,
/Must

@mustah
Copy link
Author

mustah commented Jul 5, 2012

Hi Bas. Do you want to merge my forked version with yours?

And maybe update the plugin on your account. Because there are two TabSwitch plugins now available. I can remove mine (http://plugins.intellij.net/plugin/index?pr=idea&pluginId=7062).

You can find the lastest version here: https://github.com/mustah/TabSwitch/blob/master/TabSwitch.jar

Thx,
/Must

must added 9 commits January 31, 2013 00:23
…files that where changed when upgraded to intellij idea 12
- moved class up a level from their interfaces
- extracted more descriptive methods
- removed final keyword - boiler plate code
- more fluent method calls where possible
- added my email address as bug report email address.
- compiled using new version of intellij (ipr-file has been changed)
- removed interface with only one implementation. Use extract interface if needed in the future. Use KISS!
- more fluent language to create component
- interface is not necessary here - extract one if needed
must and others added 30 commits December 18, 2013 00:23
…e with @nullable when null can be returned or passed in.
- added word do my dictionary
- do not ignore to list the popup when no files are opened. Just display that no matches where found.
- do not show when no files are open
New version of the plugin is also created. Version 3.0.6, where the
unused class is removed from the source code.
Might have found a possible bug here. Also, rename some classes and
variables.
Add "previous tab" actions, so we can trigger them directly.
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.

3 participants