1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-07-05 16:15:25 +02:00
Commit graph

54 commits

Author SHA1 Message Date
Alena Laskavaia
4b6a04fac1 launchbar: resurrect gear on target selector
it will open Properties dialog on connection.
Contributors can adapt connection to their class
to show various property pages and allow to edit "targets"

Change-Id: I0fe486b069450bbf8379930ffdeaabd9c8dbbd6e
2015-08-26 13:49:06 -04:00
Alena Laskavaia
38577c4ec9 NPE in StopActiveCommandHandler when origin is null
Change-Id: Ic6749311328d9521d64c5d339006c0065e200117
2015-08-25 09:30:27 -04:00
Alena Laskavaia
6c7b1ba259 launchbar: make launchbar selectors resizeable
Change-Id: Iba264b3b1a666257663ba36bca9262d0c6339f89
2015-07-27 10:18:58 -04:00
Alena Laskavaia
0fc265dc4e launchbar: fixes in config edit wizard page
- removed code that did nothing
- added dispose code to restore lc dialog otherwise if you cancel cannot
bring any sort of lc dialog again
- added code to have unique lc name as suggestion

Change-Id: Ia33abed6019edb097cc775f00cbfea1629e9720b
2015-07-24 08:40:55 -04:00
Alena Laskavaia
9401e51001 launchbar: fixing some labels
- mode selector tooltip Launch configuration -> Launch Mode
- launch selector Launch configuration -> Launch Configuration
- new text Create New Configuration... -> New Launch Configuration...
(consistent with tooltip for same button in lc dialog)

Change-Id: I339ce4263783a08d10f11f3474abf9f4b468e775
2015-07-24 08:40:55 -04:00
Alena Laskavaia
9986e4d6f6 launchbar: using native toolbar items for buttons
- tested on linux ubuntu 14.04 gtk 3.10, mac os x 10.10.3
and windows 7 - looks good. On linux it has extra
spacing but this is consistent with other toolbar items.

Change-Id: Iafeb1510ce83163f77a09f61c28a4f67d14ab0a1
2015-07-23 14:37:07 -04:00
Alena Laskavaia
3a6a685bdb Bug 470717 - [Launch Bar] NPE in some launch configuration when creating
through launch bar wizard

Change-Id: Ib4627341b3074613dd8e42e71e85161267a37c65
2015-06-22 10:26:10 -04:00
Alena Laskavaia
a637a8a4ef Bug 469522 - [Launch bar] An internal error occurred during: "Stopping
launches" (NPE)

Change-Id: I3d3af8182680a39cccc734015927235bb4bcc227
Signed-off-by: Alena Laskavaia <elaskavaia.cdt@gmail.com>
2015-06-22 10:26:01 -04:00
Alena Laskavaia
329f20b8c9 launchbar: NPE in ModeSelector is group does not have an image
this prevents mode selector from working at all

Change-Id: Iefc80b8a71cc4550e35fedb2dadcc450f256cb63
2015-06-19 09:56:46 -04:00
Doug Schaefer
d6a022692a Remove 32-bit Mac. It's gone now.
Change-Id: Id2b00af1eda3a5c990a4f7ec774b68628c565d71
2015-06-19 09:56:35 -04:00
Alena Laskavaia
9953b7f05d launchbar: get rid of startup warning about missing enablement expr
Change-Id: Id9cefa69f1cf532d54eb5fd18724d8ae1d50aa52
2015-06-15 09:10:24 -04:00
Alena Laskavaia
01bfa62a4d launchbar: target config provider lc working copies bugs
per target config provider adds working copy into cache on change
notifications which creates problems
a) they never cleaned up
b) now when its searching its picking a wrong config (wc) causing 
it to return invalid config

fix push up higher in launch bar itself, no point propagating
launch change notifications from working copies

Change-Id: I3b7bc94f3ef6ab385fc43cc5482ab4c9b943e8fa
2015-06-03 11:11:45 -04:00
Doug Schaefer
e1ac2000de Final LaunchBar 1.0 API. Lots of clean up and simplification.
launchConfigAdded becomes pretty important. Requires providers
to be able to find descriptor and target from the incoming
configuration if they own it. Also added handing for a default
when the target is null.

removed ownsConfiguration in providers since the launchConfigAdd
does that by returning false.

Added enablement for config providers, but that will need work
and some default property testers for it to be useful.

For the correct way to add in a config provider, check out the
Arduino change for this API at:

https://git.eclipse.org/r/#/c/49029/

Change-Id: I46b194a933c846d7d3e4eea8a0213c4ec324816f
2015-05-30 23:34:00 -04:00
Alena Laskavaia
f4944e5e27 launchbar: enablement expressions for desc. type
added ability to define enablement for descriptor type,
to allow lazy loading of plugins



Change-Id: I9a9f494d0720f752c0cfebf20316f9f8274ee23f
2015-05-29 13:16:51 -04:00
Alena Laskavaia
b46cc02560 launch bar: per target provider and test fixes
- fixed tests and added tests for previous API changes
- fixed unsafe call of provider methods without catching exceptions
- added description of priority attribute in the extension point API
- fixed default return values in DefaultLaunchConfigProvider, in case it
is extended
- removed unused import
- fixed per target provider to support persistance
- added test for per target provider

Change-Id: If08b18b939e86757108a800d1092a62621a8c7d0
2015-05-28 16:54:43 -04:00
Rob Stryker
652500b200 Bug 468399 - mode selector filled with tostrings
Change-Id: I109b728b2b7e1356c026c11b3b3663089940c5fe
Signed-off-by: Rob Stryker <rob.stryker@jboss.com>
2015-05-26 15:57:11 -04:00
Alena Laskavaia
93a926385f launch bar: API for config added/changed
we missing API to notify provider that configs are added/changed.
it required to keep internal provider map in sync, i.e. restore
the state of config/descriptor map on startup for example.

Change-Id: I3af24a21d5d41084f85731f6da2f7aca2df81f1c
2015-05-26 14:00:16 -04:00
Doug Schaefer
32a1b3134c Fix missing about files. 2015-05-21 10:21:47 -04:00
92a0b1cb0e launcbar: shash does not drag on mac
Change-Id: I89c37ad8ae0a3ea6cc3e2d5b2450fd8ed1bafa33
Signed-off-by: <elaskavaia.cdt@gmail.com>
2015-05-20 16:02:31 -04:00
Doug Schaefer
456b2f42b5 Remove the special launch config delegate for remote.
It's too hard to replicate the entire debug platform launch sequence
in order to insert the target this way. We'll need to come up with
something better if we want to share launch configs between targets.
But that will have to wait until Neon at the earliest.

Change-Id: I208eed527961e172316f0b12b869be5665a6aca5
2015-05-19 10:41:04 -04:00
Doug Schaefer
f6e8b921c3 Bug 467153 - Remove last remnants of per type config providers.
Also adds a tighter check for ownsConfiguration based on class name
of the provider that created the config.

Change-Id: If197246af0906cb5af92819171ee97dc7cd30bee
2015-05-18 23:29:06 -04:00
Doug Schaefer
86c73092fa Bug 467153 - Step one of making configs per target.
WTP needs a new config for every target. That breaks our architecture
of one config per target type. But that's actually already broken
since we can't really get the proper target to the launch config.
Depending on active, we always hoped the target didn't change, but
it can. So one config per target is correct anyway.

Change-Id: I14ba8413b9494a13f3496eed5535debbc330a56c
2015-05-17 23:22:09 -04:00
Rob Stryker
899208ac28 Bug 466488 - Clicking edit on launches that have no launch group, tabs,
or have other errors, causes several error dialogs to pop up.

Change-Id: Id5b56c2696e4911143a4daeb93cc8a442b3ebe33
Signed-off-by: Rob Stryker <rob.stryker@jboss.com>
2015-05-07 14:29:11 -04:00
Alena Laskavaia
d7a91f70ee Bug 466490: [launchbar] Color of the sash
Trying to make sash more visible

Change-Id: I6baeb1b05451e808c86204d0b0b1a1c13af5c7de
2015-05-07 08:31:39 -04:00
Rob Stryker
4e67214de3 Bug 466489 - Limit launchbar-visible configs to public launch types; fix unit test
- Cleanup of bad catch block

Change-Id: If5c9adcc6970ffb480312e2453fcb153674cf79b
Signed-off-by: Rob Stryker <rob.stryker@jboss.com>
2015-05-06 09:37:16 -04:00
Alena Laskavaia
fd58fed7e4 launch bar: corrected edit dialog handling of adding of new LC
- launch bar launch config was not handling add of another launch config
correctly - it was just resetting itself to handle new LC. This cannot
really be done because this dialog is bound to launch group and new lc
may not be part of it and may not even support the mode current dialog
is in. Also that make missyncronization between LC stored in parent and
LC stored in getTabViewer which can cause other nasty problems.
So let parent handle adding (which only handles locatio change) and for
duplication we just open a new dialog and close current one.

Change-Id: I2dde5c3af26d901d1f4b18cad84a83a9857a6ca9
2015-05-05 16:33:56 -04:00
Alena Laskavaia
03ab4beaf8 launch bar: "launch" button name in LC edit dialog
- use same name as in LaunchConfigurationDialog, i.e. Run, Debug, etc.

Change-Id: I497ff5c7913c47e3cfeb736557a3c8e90040d2ef
2015-05-05 11:19:25 -04:00
Alena Laskavaia
3e548fced4 launchbar: hook up connect/disconnect state for target selector
Change-Id: I93240fa2385053454fe9f84279eddc1518290d4f
2015-05-04 13:44:16 -04:00
Doug Schaefer
3c33b772b8 Bug 466095 - Externalize Strings and minimize warnings.
Removed dead code and minimize use of internal interfaces.

Change-Id: Ief9c29bff721940bf5a173dfacf60b715691dcf7
2015-05-01 13:42:54 -04:00
Doug Schaefer
e8eec12ec4 Bug 466089 - Stop only launches for the active config.
Turns out to be pretty easy, just check the config for each launch
and stop the ones that match the active config.

Change-Id: Ib74a31405382b2d1996b257f7e8ad1d85a23e6c6
2015-05-01 12:21:04 -04:00
Alena Laskavaia
f2b778e3c7 launchbar: active remote connection is not persisted in the remote
services

- if you select target for given launch config and restart IDE
it is not persistent, because it queries remote services and
it has not been set. Moving setting of target in the correct place
Note: if launch config is not created yet it would not be create
and mapping would not be set but it would be persisted by launch bar
itself
- also added protection against null listener

Change-Id: I41f8651ba2251ca9f841450fc1f511643bfde681
2015-04-27 16:05:04 -04:00
Alena Laskavaia
013c6fa1fe launch-bar: prevent event storm on initialization
- prevent events dispatch during initialization of launch bar manager
- also fix case when connection is renamed - we don't need to remove it

Change-Id: I8ba5938afd21c9b2459563cd9e3385fb9a69f673
2015-04-10 12:59:39 -04:00
Alena Laskavaia
baa294b085 launch-bar: config selector - remove active config chage
in last patch I did not actually remove setting active
config from configSelector - it should not set it, but
it will be set automatically when config is saved.


Change-Id: Idcb84d2d71567f1e60cb452fb4df15ed962dded6
2015-04-10 10:00:16 -04:00
Alena Laskavaia
17fff835d1 launch-bar: added some guards for mode selection and API for setting
mode

* added API for setting mode and target without changing active
descriptor
* used this API in the config selector to set the mode
* added guard against mismatched mode


Change-Id: I58da4aac12950c21192385820683623f9e939786
Signed-off-by: Alena Laskavaia <elaskavaia.cdt@gmail.com>
2015-04-09 11:39:56 -04:00
Alena Laskavaia
28985bd18d launch bar: renamed test so it picked by gerrit verifier
- also mocked another target so it does not always test local one

Change-Id: Idf59342aac75bf5c92c5ca63870eade760065f70
Signed-off-by: Alena Laskavaia <elaskavaia.cdt@gmail.com>
2015-03-18 16:28:55 -04:00
Alena Laskavaia
b447d528f8 launch bar: tests restored and test fixes
- restored test module in pom
- fixed tests - xml tag was renamed
- fixed tests - stub functionality that rely on o.e.remote
- fixed NPE when accesssing osname from LaunchTargetTypeInfo
- added guards when calling into user defined classes 
- added more comments in java docs
- restored original launch bar tests
- current provider won't get notified when lc is removed, need to fix
more

Change-Id: I15f1a30e9f965ad5c84bdc51238ef129dc314450
Signed-off-by: Alena Laskavaia <elaskavaia.cdt@gmail.com>
2015-03-18 15:24:10 -04:00
Alena Laskavaia
e1fdb68ea3 launchbar: command to open config selector plus more
- added command to open config selector (unbound now)
- refactored common code to edit config (move into edit config command)
- removed unused includes
- changed wording on other launch bar command for consistency

Change-Id: I9af26cc307cfde4e63d8caa3822a169f42638ef3
2015-03-17 11:29:36 -04:00
Doug Schaefer
210058b955 Hook up the new connection wizard the target selector action area.
Also externalize strings for the target selector.

Change-Id: Ib082066560a1ab786783d47f165d7638189d338d
2015-03-02 21:24:57 -05:00
Doug Schaefer
9c7de82238 Bug 459972 - Update LaunchBar to use IRemoteConnection for targets.
ILaunchTarget is removed and replaced with IRemoteConnection. You
still have to declare a launch target type to point at a connection
type. It uses os and arch to help decide what toolchains to use for
builds.

Change-Id: I8f21b4e5043ccd8af85be91c643f58ad301c3ac4
2015-02-16 02:17:27 -05:00
Doug Schaefer
5f3b1af3f1 Remove the o.e.remote proposal. It's going live now at home.
Not sure why this stuff was on master anyway. It was supposed to be
on the oeremote branch. At any rate, I've started to contribute the
new API to o.e.remote properly so this is no longer needed.

Change-Id: I83409423ee5d5ad1295efadabb9d686b4d0fce4f
2015-01-27 20:27:19 -05:00
Alena Laskavaia
457fd76f5d remove debug output
Change-Id: If587d17f8670a6f9bc2623a72c0395f921495eee
2015-01-21 13:57:22 -05:00
Alena Laskavaia
3487f9e9e2 LaunchBar: prevent widget is disposed exception on shutdown
Change-Id: I79afcded38a64a9c2f2ce0190eee1766a9d4265e
2015-01-20 15:45:14 -05:00
Alena Laskavaia
faf222b8f2 Launch Bar: ui tweaks for proper re-sizing and keyboard events
- fixed issues when list is not drawn properly
- fixed that Esc does not work to close pop up sometimes
- fixed filtering for list with edit button
- fixed layout issues when only one item in the list and filter becomes
visible (filter is disabled in this case)
Change-Id: I562f12df8326b4efecf75150b57b699c53d398da
2015-01-19 16:21:31 -05:00
Doug Schaefer
40b6b6c05c Merge "LaunchBar: prevent repaint storm during initialization" 2015-01-14 11:58:05 -05:00
Doug Schaefer
5503a7526f Merge "LaunchBar: do not allow to Finish new config wizard prematurely" 2015-01-14 11:56:48 -05:00
Alena Laskavaia
7983b686f0 LaunchBar: prevent repaint storm during initialization
when launch descriptor (and other objects) are changed
using API it can create event storm which leaves control
changing rapidly. To prevent that added X ms delay
if API is used to change launch bar ui (or during initialization)

Change-Id: Ic02587cbd89c754abd6020cac2754e971ad50a37
2015-01-14 11:24:26 -05:00
Alena Laskavaia
d4621c4aa9 LaunchBar: do not allow to Finish new config wizard prematurely
Finish button is active when not all parameters for lauch
configuration are set

Change-Id: Iae33a1679e043257bb591ae18a7db8310a93cf52
2015-01-14 09:27:26 -05:00
Alena Laskavaia
edeec7aaaa LaunchBar: ui changes in selector
- added ability to filter by typing in selector pop-up
- refactored list part of selection in its own components which is kind
of list viewer
- filter is visible by default if we have more than 7 elements in drop
down
- filter become visible if user start typing, even if it was not visible
originally
- moving storing of history size into the list viewer itself
- hooked up proper focus support and keyboard traversal
- modified CButton so we can re-use it for Edit button 
of selectors and list items
- when filtering is activated history list is no longer available

Change-Id: I2057ad658f5a5c9f8fbce203748d26d7e4b14dd6
2015-01-13 16:48:24 -05:00
Doug Schaefer
bb7f665007 Add tests and make them run (adding hamcrest properly)
Change-Id: Ifc2e5d9cd4289147539d6b90b59f689ffdf69a37
2015-01-13 14:14:56 -05:00
Alena Laskavaia
ac02204fde launchBar: sash for increasing recently used size
launch element drop down has hardcoded list of 3 elements
of recent history, sometimes it is too many, sometimes not enought.
added sash so user can resize it

save this as preference, but no ui to edit this preference in pref pages
for now
2015-01-06 16:52:46 -05:00