* [EGIT] [PATCH v1 0/1] Add an ignored icon @ 2009-02-23 19:03 Ferry Huberts 2009-02-23 19:03 ` [EGIT] [PATCH v1 1/1] " Ferry Huberts 0 siblings, 1 reply; 9+ messages in thread From: Ferry Huberts @ 2009-02-23 19:03 UTC (permalink / raw) To: git; +Cc: Ferry Huberts Add an ignored icon to the label decorations page and make sure that it is actually decorated: from now on do not ignore ignored resources during decoration. Ferry Huberts (1): Add an ignored icon org.spearce.egit.ui/icons/ovr/ignored.gif | Bin 0 -> 102 bytes .../egit/ui/PluginPreferenceInitializer.java | 1 + .../src/org/spearce/egit/ui/UIIcons.java | 4 ++++ .../src/org/spearce/egit/ui/UIPreferences.java | 2 ++ .../src/org/spearce/egit/ui/UIText.java | 3 +++ .../decorators/GitLightweightDecorator.java | 11 ++++++++--- .../preferences/GitDecoratorPreferencePage.java | 12 ++++++++++++ .../src/org/spearce/egit/ui/uitext.properties | 1 + 8 files changed, 31 insertions(+), 3 deletions(-) create mode 100644 org.spearce.egit.ui/icons/ovr/ignored.gif ^ permalink raw reply [flat|nested] 9+ messages in thread
* [EGIT] [PATCH v1 1/1] Add an ignored icon 2009-02-23 19:03 [EGIT] [PATCH v1 0/1] Add an ignored icon Ferry Huberts @ 2009-02-23 19:03 ` Ferry Huberts 2009-02-23 23:59 ` Tor Arne Vestbø 0 siblings, 1 reply; 9+ messages in thread From: Ferry Huberts @ 2009-02-23 19:03 UTC (permalink / raw) To: git; +Cc: Ferry Huberts Add an ignored icon to the label decorations page and make sure that it is actually decorated: from now on do not ignore ignored resources during decoration. Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl> --- org.spearce.egit.ui/icons/ovr/ignored.gif | Bin 0 -> 102 bytes .../egit/ui/PluginPreferenceInitializer.java | 1 + .../src/org/spearce/egit/ui/UIIcons.java | 4 ++++ .../src/org/spearce/egit/ui/UIPreferences.java | 2 ++ .../src/org/spearce/egit/ui/UIText.java | 3 +++ .../decorators/GitLightweightDecorator.java | 11 ++++++++--- .../preferences/GitDecoratorPreferencePage.java | 12 ++++++++++++ .../src/org/spearce/egit/ui/uitext.properties | 1 + 8 files changed, 31 insertions(+), 3 deletions(-) create mode 100644 org.spearce.egit.ui/icons/ovr/ignored.gif diff --git a/org.spearce.egit.ui/icons/ovr/ignored.gif b/org.spearce.egit.ui/icons/ovr/ignored.gif new file mode 100644 index 0000000000000000000000000000000000000000..509e706b3c847697c8fab379bc8bacdc66d21f3c GIT binary patch literal 102 zcmZ?wbhEHb<Y3@nc+AJ}>(@D15q3p!E=^fpC2=k-SzZ$rK|38WV-+D!6DfNgu|P{% zPh+VN3xoe)p!k!8k%57qL5BedKxQzo2!Gh?Hha$0M=_26udo+g4fC2Q!pL9^04%B; A)&Kwi literal 0 HcmV?d00001 diff --git a/org.spearce.egit.ui/src/org/spearce/egit/ui/PluginPreferenceInitializer.java b/org.spearce.egit.ui/src/org/spearce/egit/ui/PluginPreferenceInitializer.java index 8d617e9..f76b6e1 100644 --- a/org.spearce.egit.ui/src/org/spearce/egit/ui/PluginPreferenceInitializer.java +++ b/org.spearce.egit.ui/src/org/spearce/egit/ui/PluginPreferenceInitializer.java @@ -46,6 +46,7 @@ public void initializeDefaultPreferences() { UIText.DecoratorPreferencesPage_projectFormatDefault); prefs.setDefault(UIPreferences.DECORATOR_SHOW_TRACKED_ICON, true); prefs.setDefault(UIPreferences.DECORATOR_SHOW_UNTRACKED_ICON, true); + prefs.setDefault(UIPreferences.DECORATOR_SHOW_IGNORED_ICON, true); prefs.setDefault(UIPreferences.DECORATOR_SHOW_STAGED_ICON, true); prefs.setDefault(UIPreferences.DECORATOR_SHOW_CONFLICTS_ICON, true); prefs.setDefault(UIPreferences.DECORATOR_SHOW_ASSUME_VALID_ICON, true); diff --git a/org.spearce.egit.ui/src/org/spearce/egit/ui/UIIcons.java b/org.spearce.egit.ui/src/org/spearce/egit/ui/UIIcons.java index 952816c..5bc1eaf 100644 --- a/org.spearce.egit.ui/src/org/spearce/egit/ui/UIIcons.java +++ b/org.spearce.egit.ui/src/org/spearce/egit/ui/UIIcons.java @@ -31,6 +31,9 @@ /** Decoration for resource not being tracked by Git */ public static final ImageDescriptor OVR_UNTRACKED; + /** Decoration for resource ignored by Git */ + public static final ImageDescriptor OVR_IGNORED; + /** Decoration for tracked resource with a merge conflict. */ public static final ImageDescriptor OVR_CONFLICT; @@ -92,6 +95,7 @@ OVR_STAGED_ADD = map("ovr/staged_added.gif"); OVR_STAGED_REMOVE = map("ovr/staged_removed.gif"); OVR_UNTRACKED = map("ovr/untracked.gif"); + OVR_IGNORED = map("ovr/ignored.gif"); OVR_CONFLICT = map("ovr/conflict.gif"); OVR_ASSUMEVALID = map("ovr/assume_valid.gif"); ELCL16_FIND = map("elcl16/find.gif"); diff --git a/org.spearce.egit.ui/src/org/spearce/egit/ui/UIPreferences.java b/org.spearce.egit.ui/src/org/spearce/egit/ui/UIPreferences.java index bbc36be..cdf9ab4 100644 --- a/org.spearce.egit.ui/src/org/spearce/egit/ui/UIPreferences.java +++ b/org.spearce.egit.ui/src/org/spearce/egit/ui/UIPreferences.java @@ -67,6 +67,8 @@ /** */ public final static String DECORATOR_SHOW_UNTRACKED_ICON = "decorator_show_untracked_icon"; /** */ + public final static String DECORATOR_SHOW_IGNORED_ICON = "decorator_show_ignored_icon"; + /** */ public final static String DECORATOR_SHOW_STAGED_ICON = "decorator_show_staged_icon"; /** */ public final static String DECORATOR_SHOW_CONFLICTS_ICON = "decorator_show_conflicts_icon"; diff --git a/org.spearce.egit.ui/src/org/spearce/egit/ui/UIText.java b/org.spearce.egit.ui/src/org/spearce/egit/ui/UIText.java index 0740f09..85bb50b 100644 --- a/org.spearce.egit.ui/src/org/spearce/egit/ui/UIText.java +++ b/org.spearce.egit.ui/src/org/spearce/egit/ui/UIText.java @@ -1013,6 +1013,9 @@ public static String DecoratorPreferencesPage_iconsShowUntracked; /** */ + public static String DecoratorPreferencesPage_iconsShowIgnored; + + /** */ public static String DecoratorPreferencesPage_iconsShowStaged; /** */ diff --git a/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/decorators/GitLightweightDecorator.java b/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/decorators/GitLightweightDecorator.java index 6f69c80..06c10e1 100644 --- a/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/decorators/GitLightweightDecorator.java +++ b/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/decorators/GitLightweightDecorator.java @@ -224,6 +224,8 @@ public ImageData getImageData() { private static ImageDescriptor untrackedImage; + private static ImageDescriptor ignoredImage; + private static ImageDescriptor stagedImage; private static ImageDescriptor stagedAddedImage; @@ -238,6 +240,7 @@ public ImageData getImageData() { trackedImage = new CachedImageDescriptor(TeamImages .getImageDescriptor(ISharedImages.IMG_CHECKEDIN_OVR)); untrackedImage = new CachedImageDescriptor(UIIcons.OVR_UNTRACKED); + ignoredImage = new CachedImageDescriptor(UIIcons.OVR_IGNORED); stagedImage = new CachedImageDescriptor(UIIcons.OVR_STAGED); stagedAddedImage = new CachedImageDescriptor(UIIcons.OVR_STAGED_ADD); stagedRemovedImage = new CachedImageDescriptor( @@ -269,9 +272,6 @@ public DecorationHelper(IPreferenceStore preferencesStore) { */ public void decorate(IDecoration decoration, IDecoratableResource resource) { - if (resource.isIgnored()) - return; - decorateText(decoration, resource); decorateIcons(decoration, resource); } @@ -335,6 +335,11 @@ else if (staged == Staged.REMOVED) && resource.hasConflicts()) overlay = conflictImage; + } else if (resource.isIgnored()) { + if (store + .getBoolean(UIPreferences.DECORATOR_SHOW_IGNORED_ICON)) { + overlay = ignoredImage; + } } else if (store .getBoolean(UIPreferences.DECORATOR_SHOW_UNTRACKED_ICON)) { overlay = untrackedImage; diff --git a/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/preferences/GitDecoratorPreferencePage.java b/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/preferences/GitDecoratorPreferencePage.java index eca2277..8c950cf 100644 --- a/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/preferences/GitDecoratorPreferencePage.java +++ b/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/preferences/GitDecoratorPreferencePage.java @@ -99,6 +99,8 @@ private Button showUntracked; + private Button showIgnored; + private Preview preview; private Button showStaged; @@ -308,6 +310,8 @@ private Control createIconDecoratorPage(Composite parent) { UIText.DecoratorPreferencesPage_iconsShowTracked); showUntracked = SWTUtils.createCheckBox(imageGroup, UIText.DecoratorPreferencesPage_iconsShowUntracked); + showIgnored = SWTUtils.createCheckBox(imageGroup, + UIText.DecoratorPreferencesPage_iconsShowIgnored); showStaged = SWTUtils.createCheckBox(imageGroup, UIText.DecoratorPreferencesPage_iconsShowStaged); showConflicts = SWTUtils.createCheckBox(imageGroup, @@ -376,6 +380,8 @@ private void initializeValues() { .getBoolean(UIPreferences.DECORATOR_SHOW_TRACKED_ICON)); showUntracked.setSelection(store .getBoolean(UIPreferences.DECORATOR_SHOW_UNTRACKED_ICON)); + showIgnored.setSelection(store + .getBoolean(UIPreferences.DECORATOR_SHOW_IGNORED_ICON)); showStaged.setSelection(store .getBoolean(UIPreferences.DECORATOR_SHOW_STAGED_ICON)); showConflicts.setSelection(store @@ -391,6 +397,7 @@ public void widgetSelected(SelectionEvent e) { showTracked.addSelectionListener(selectionListener); showUntracked.addSelectionListener(selectionListener); + showIgnored.addSelectionListener(selectionListener); showStaged.addSelectionListener(selectionListener); showConflicts.addSelectionListener(selectionListener); showAssumeValid.addSelectionListener(selectionListener); @@ -447,6 +454,8 @@ private boolean performOk(IPreferenceStore store) { .getSelection()); store.setValue(UIPreferences.DECORATOR_SHOW_UNTRACKED_ICON, showUntracked.getSelection()); + store.setValue(UIPreferences.DECORATOR_SHOW_IGNORED_ICON, + showIgnored.getSelection()); store.setValue(UIPreferences.DECORATOR_SHOW_STAGED_ICON, showStaged .getSelection()); store.setValue(UIPreferences.DECORATOR_SHOW_CONFLICTS_ICON, @@ -485,6 +494,9 @@ protected void performDefaults() { showUntracked .setSelection(store .getDefaultBoolean(UIPreferences.DECORATOR_SHOW_UNTRACKED_ICON)); + showIgnored + .setSelection(store + .getDefaultBoolean(UIPreferences.DECORATOR_SHOW_IGNORED_ICON)); showStaged.setSelection(store .getDefaultBoolean(UIPreferences.DECORATOR_SHOW_STAGED_ICON)); showConflicts diff --git a/org.spearce.egit.ui/src/org/spearce/egit/ui/uitext.properties b/org.spearce.egit.ui/src/org/spearce/egit/ui/uitext.properties index 847cb2c..932c75c 100644 --- a/org.spearce.egit.ui/src/org/spearce/egit/ui/uitext.properties +++ b/org.spearce.egit.ui/src/org/spearce/egit/ui/uitext.properties @@ -376,6 +376,7 @@ DecoratorPreferencesPage_textLabel=T&ext Decorations DecoratorPreferencesPage_iconLabel=&Icon Decorations DecoratorPreferencesPage_iconsShowTracked=Tracked resources DecoratorPreferencesPage_iconsShowUntracked=Untracked resources +DecoratorPreferencesPage_iconsShowIgnored=Ignored resources DecoratorPreferencesPage_iconsShowStaged=Staged resources DecoratorPreferencesPage_iconsShowConflicts=Conflicting resources DecoratorPreferencesPage_iconsShowAssumeValid=Assumed unchanged resources -- 1.6.0.6 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [EGIT] [PATCH v1 1/1] Add an ignored icon 2009-02-23 19:03 ` [EGIT] [PATCH v1 1/1] " Ferry Huberts @ 2009-02-23 23:59 ` Tor Arne Vestbø 2009-02-24 5:15 ` Ferry Huberts (Pelagic) 2009-02-24 7:06 ` Robin Rosenberg 0 siblings, 2 replies; 9+ messages in thread From: Tor Arne Vestbø @ 2009-02-23 23:59 UTC (permalink / raw) To: Ferry Huberts; +Cc: git, Robin Rosenberg, Shawn O. Pearce Ferry Huberts wrote: > Add an ignored icon to the label decorations page and make > sure that it is actually decorated: from now on do not ignore > ignored resources during decoration. The reason this was not added in the original series was because that's kind of the point of ignoring a resource -- you don't want any information about it. Also, none of the other team plugins provide decorations for ignored resources- On the other hand, since the decorators are now user configurable, adding the option to allow users to enable decoration of ignored resources if they really feel it's useful to them is admittedly in line with the whole customization idea. Though I'm not sure if this particular case of customization would cause more confusion than good, for example in terms of which doors we open by allowing ignored resources to actually not be ignored (i.e what other features would you expect to work for ignored resources?). > prefs.setDefault(UIPreferences.DECORATOR_SHOW_UNTRACKED_ICON, true); > + prefs.setDefault(UIPreferences.DECORATOR_SHOW_IGNORED_ICON, true); > prefs.setDefault(UIPreferences.DECORATOR_SHOW_STAGED_ICON, true); If applied, I would argue that the option should be _off_ by default, to match the behavior of the other team plugins and the normal logical action of ignoring a resource. > + /** Decoration for resource ignored by Git */ > + public static final ImageDescriptor OVR_IGNORED; > + Side-note: the current decoration implementation only uses Eclipse's concept of an ignored resource -- not .gitignore et al. since we don't have a standard way of reading those yet. Tor Arne ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [EGIT] [PATCH v1 1/1] Add an ignored icon 2009-02-23 23:59 ` Tor Arne Vestbø @ 2009-02-24 5:15 ` Ferry Huberts (Pelagic) 2009-02-24 6:11 ` Ferry Huberts (Pelagic) 2009-02-24 7:06 ` Robin Rosenberg 1 sibling, 1 reply; 9+ messages in thread From: Ferry Huberts (Pelagic) @ 2009-02-24 5:15 UTC (permalink / raw) To: Tor Arne Vestbø; +Cc: git, Robin Rosenberg, Shawn O. Pearce Tor Arne Vestbø wrote: > Ferry Huberts wrote: >> Add an ignored icon to the label decorations page and make >> sure that it is actually decorated: from now on do not ignore >> ignored resources during decoration. > > The reason this was not added in the original series was because that's > kind of the point of ignoring a resource -- you don't want any > information about it. Also, none of the other team plugins provide > decorations for ignored resources- > I could also argue that I want everything to be explicit, even the status of ignored files :-) If you look at (for example) TortoiseSVN then you'll see that it does provide an ignore icon. My icon was sort of 'inspired' by that one. Don't know if it's on by default though, will have to check that. > On the other hand, since the decorators are now user configurable, > adding the option to allow users to enable decoration of ignored > resources if they really feel it's useful to them is admittedly in line > with the whole customization idea. > > Though I'm not sure if this particular case of customization would cause > more confusion than good, for example in terms of which doors we open by > allowing ignored resources to actually not be ignored (i.e what other > features would you expect to work for ignored resources?). > >> prefs.setDefault(UIPreferences.DECORATOR_SHOW_UNTRACKED_ICON, true); >> + prefs.setDefault(UIPreferences.DECORATOR_SHOW_IGNORED_ICON, true); >> prefs.setDefault(UIPreferences.DECORATOR_SHOW_STAGED_ICON, true); > > If applied, I would argue that the option should be _off_ by default, to > match the behavior of the other team plugins and the normal logical > action of ignoring a resource. not my preference (hey I wrote the patch) but reasonable. > >> + /** Decoration for resource ignored by Git */ >> + public static final ImageDescriptor OVR_IGNORED; >> + > > Side-note: the current decoration implementation only uses Eclipse's > concept of an ignored resource -- not .gitignore et al. since we don't > have a standard way of reading those yet. > > Tor Arne > I've started work on that, see the issue on the issue tracker. The plan is to hook in on change events and then parse all gitignore files, and tell eclipse about the contents of the ignores. Ferry ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [EGIT] [PATCH v1 1/1] Add an ignored icon 2009-02-24 5:15 ` Ferry Huberts (Pelagic) @ 2009-02-24 6:11 ` Ferry Huberts (Pelagic) 2009-02-24 9:23 ` Tor Arne Vestbø 0 siblings, 1 reply; 9+ messages in thread From: Ferry Huberts (Pelagic) @ 2009-02-24 6:11 UTC (permalink / raw) To: git; +Cc: Tor Arne Vestbø, Robin Rosenberg, Shawn O. Pearce On Tue, February 24, 2009 06:15, Ferry Huberts (Pelagic) wrote: > Tor Arne Vestbø wrote: >> Ferry Huberts wrote: >>> Add an ignored icon to the label decorations page and make >>> sure that it is actually decorated: from now on do not ignore >>> ignored resources during decoration. >> >> The reason this was not added in the original series was because that's >> kind of the point of ignoring a resource -- you don't want any >> information about it. Also, none of the other team plugins provide >> decorations for ignored resources- >> > > I could also argue that I want everything to be explicit, even the > status of ignored files :-) > > If you look at (for example) TortoiseSVN then you'll see that it does > provide an ignore icon. My icon was sort of 'inspired' by that one. > Don't know if it's on by default though, will have to check that. I just checked, TortoiseSVN has the overlay for ignored items on by default ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [EGIT] [PATCH v1 1/1] Add an ignored icon 2009-02-24 6:11 ` Ferry Huberts (Pelagic) @ 2009-02-24 9:23 ` Tor Arne Vestbø 2009-02-24 9:22 ` Ferry Huberts (Pelagic) 0 siblings, 1 reply; 9+ messages in thread From: Tor Arne Vestbø @ 2009-02-24 9:23 UTC (permalink / raw) To: Ferry Huberts (Pelagic); +Cc: git, Robin Rosenberg, Shawn O. Pearce Ferry Huberts (Pelagic) wrote: > On Tue, February 24, 2009 06:15, Ferry Huberts (Pelagic) wrote: >> Tor Arne Vestbø wrote: >>> Ferry Huberts wrote: >>>> Add an ignored icon to the label decorations page and make >>>> sure that it is actually decorated: from now on do not ignore >>>> ignored resources during decoration. >>> The reason this was not added in the original series was because that's >>> kind of the point of ignoring a resource -- you don't want any >>> information about it. Also, none of the other team plugins provide >>> decorations for ignored resources- >>> >> I could also argue that I want everything to be explicit, even the >> status of ignored files :-) That's a reasonable argument :) I'm not really opposed to the feature -- I was just trying to think of reasons why it might not be such as good idea, just to be safe and have that discussion. But you are right. Acked-by: Tor Arne Vestbø <torarnv@gmail.com> >> If you look at (for example) TortoiseSVN then you'll see that it does >> provide an ignore icon. My icon was sort of 'inspired' by that one. >> Don't know if it's on by default though, will have to check that. > > I just checked, TortoiseSVN has the overlay for ignored items on by default The difference is TortoiseSVN does not come with a set of global default ignored files (as far as I know). Eclipse does, see Team->Ignored... So, a globally ignored file such as foobar.BAK would show up as non-decorated if shared using the CVS or SVN plugins, but with an icon if shared with Git. That might be confusing to users, especially since the minus-sign icon does not have any history in Eclipse as something being "ignored" (it actually breaks the convention of using non-decoration -- meaning no "untracked"-icon -- to signal ignored). So, I would still argue that we should leave it off by default. Tor Arne ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [EGIT] [PATCH v1 1/1] Add an ignored icon 2009-02-24 9:23 ` Tor Arne Vestbø @ 2009-02-24 9:22 ` Ferry Huberts (Pelagic) 0 siblings, 0 replies; 9+ messages in thread From: Ferry Huberts (Pelagic) @ 2009-02-24 9:22 UTC (permalink / raw) To: Tor Arne Vestbø; +Cc: Ferry Huberts, git, Robin Rosenberg, Shawn O. Pearce >> I just checked, TortoiseSVN has the overlay for ignored items on by default > > The difference is TortoiseSVN does not come with a set of global default > ignored files (as far as I know). Eclipse does, see Team->Ignored... > > So, a globally ignored file such as foobar.BAK would show up as > non-decorated if shared using the CVS or SVN plugins, but with an icon > if shared with Git. That might be confusing to users, especially since > the minus-sign icon does not have any history in Eclipse as something > being "ignored" (it actually breaks the convention of using > non-decoration -- meaning no "untracked"-icon -- to signal ignored). > > So, I would still argue that we should leave it off by default. > For now I think that's acceptable. However, when I complete the work on the ignores then we should switch it on: I plan to let EGit NOT look at the Eclipse global ignore list because the ignores are defined by .gitignore files for git repositories. At that point it totally makes sense to decorate ignored resources because these are configured in .gitignore files ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [EGIT] [PATCH v1 1/1] Add an ignored icon 2009-02-23 23:59 ` Tor Arne Vestbø 2009-02-24 5:15 ` Ferry Huberts (Pelagic) @ 2009-02-24 7:06 ` Robin Rosenberg 2009-02-24 9:28 ` Tor Arne Vestbø 1 sibling, 1 reply; 9+ messages in thread From: Robin Rosenberg @ 2009-02-24 7:06 UTC (permalink / raw) To: Tor Arne Vestbø; +Cc: Ferry Huberts, git, Shawn O. Pearce tisdag 24 februari 2009 00:59:30 skrev Tor Arne Vestbø <torarnv@gmail.com>: > Ferry Huberts wrote: > > Add an ignored icon to the label decorations page and make > > sure that it is actually decorated: from now on do not ignore > > ignored resources during decoration. > > The reason this was not added in the original series was because that's > kind of the point of ignoring a resource -- you don't want any > information about it. Also, none of the other team plugins provide > decorations for ignored resources- "Ignored" in this context does not mean unimportant. It means we will not track it in git, which is quite important. -- robin ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [EGIT] [PATCH v1 1/1] Add an ignored icon 2009-02-24 7:06 ` Robin Rosenberg @ 2009-02-24 9:28 ` Tor Arne Vestbø 0 siblings, 0 replies; 9+ messages in thread From: Tor Arne Vestbø @ 2009-02-24 9:28 UTC (permalink / raw) To: Robin Rosenberg; +Cc: Ferry Huberts, git, Shawn O. Pearce Robin Rosenberg wrote: > tisdag 24 februari 2009 00:59:30 skrev Tor Arne Vestbø <torarnv@gmail.com>: >> Ferry Huberts wrote: >>> Add an ignored icon to the label decorations page and make >>> sure that it is actually decorated: from now on do not ignore >>> ignored resources during decoration. >> The reason this was not added in the original series was because that's >> kind of the point of ignoring a resource -- you don't want any >> information about it. Also, none of the other team plugins provide >> decorations for ignored resources- > > "Ignored" in this context does not mean unimportant. It means we will > not track it in git, which is quite important. True, it's important. Untracked already has a decoration (the '?'), and tracked files have the yellow database icon. I'm just saying the convention so far in Eclipse has been to use non-decoration (no ? or yellow db icon) to signal ignored resources. Tor Arne ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2009-02-24 9:24 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2009-02-23 19:03 [EGIT] [PATCH v1 0/1] Add an ignored icon Ferry Huberts 2009-02-23 19:03 ` [EGIT] [PATCH v1 1/1] " Ferry Huberts 2009-02-23 23:59 ` Tor Arne Vestbø 2009-02-24 5:15 ` Ferry Huberts (Pelagic) 2009-02-24 6:11 ` Ferry Huberts (Pelagic) 2009-02-24 9:23 ` Tor Arne Vestbø 2009-02-24 9:22 ` Ferry Huberts (Pelagic) 2009-02-24 7:06 ` Robin Rosenberg 2009-02-24 9:28 ` Tor Arne Vestbø
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).
