The Inbox: Monticello-pre.622.mcz

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

The Inbox: Monticello-pre.622.mcz

commits-2
A new version of Monticello was added to project The Inbox:
http://source.squeak.org/inbox/Monticello-pre.622.mcz

==================== Summary ====================

Name: Monticello-pre.622
Author: pre
Time: 11 November 2015, 3:13:40.127 pm
UUID: 59ac83cc-5186-4844-bb8e-1c1009157e88
Ancestors: Monticello-cmm.621

Adds a keyboard shortcut for reverting changes to Monticello browsers which are showing changes. This should allow easy reversion of changes shown in diffs.

=============== Diff against Monticello-cmm.621 ===============

Item was added:
+ ----- Method: MCOperationsBrowser>>methodListKey:from: (in category 'menus') -----
+ methodListKey: aKeystroke from: aListMorph
+ aKeystroke caseOf: {
+ [$x] -> [self revertSelection] }
+ otherwise: [super methodListKey: aKeystroke from: aListMorph ]!

Item was changed:
  ----- Method: MCOperationsBrowser>>methodListMenu: (in category 'menus') -----
  methodListMenu: aMenu
  selection ifNotNil:
+ [aMenu addList: #(
- [aMenu addList:#(
  ('install' installSelection)
+ ('revert (x)' revertSelection)
+ ('browse origin' browseSelectionOrigin)
- ('revert' revertSelection)
- ('browse origin' browseSelectionOrigin)
  -)].
  self unchangedMethods ifNotEmpty:
+ [aMenu addList: #(
+ ('revert unchanged methods...' revertUnchangedMethods)
- [aMenu addList:#(
- ('revert unchanged methods...' revertUnchangedMethods)
  -)].
  super methodListMenu: aMenu.
  ^ aMenu!


Reply | Threaded
Open this post in threaded view
|

Re: The Inbox: Monticello-pre.622.mcz

Patrick R.
A short note on why I have used the $x shortcut instead of $r: The $r is already used by the MCMergeBrowser to reject new changes and at the same time $x for reverting matches the remove semantic.

The $r and $k shortcuts of the MergeBrowser are not yet presented to users in any way. This should maybe be added to the MCMergeBrowser>>#innerButtonSpecs method.

Bests
Patrick