Skip to content

INT-591 only hide *.system.* collections #160

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

Merged
merged 3 commits into from
Oct 15, 2015
Merged

Conversation

rueckstiess
Copy link
Contributor

also makes specialish collections darker. (please review @marcgurney)

One small issue I found with this: If a database only contains a system.* collection (e.g. system.indexes), because all other real collections were deleted, it counts towards the DB count but does not show up in the collection list.

I wouldn't change the DB count though. The number it shows is correct, and anyone running "show dbs" will get the same count.

screen shot 2015-10-15 at 12 30 22 pm

@rueckstiess
Copy link
Contributor Author

Also added a title tip for all collections so that too long names can still be shown.

The special collections that we are not filtering out now also have "(internal collection)" added in the title tip.

screen shot 2015-10-15 at 1 22 19 pm

@rueckstiess rueckstiess changed the title 🚧 INT-591 only hide *.system.* collections INT-591 only hide *.system.* collections Oct 15, 2015
@marcgurney
Copy link
Contributor

@rueckstiess switched to @gray3 instead of darkening @gray5

Removed color change on hover for special collections.

lgtm

@imlucas
Copy link
Contributor

imlucas commented Oct 15, 2015

:shipit:

imlucas added a commit that referenced this pull request Oct 15, 2015
INT-591 only hide `*.system.*` collections
@imlucas imlucas merged commit 3a60de3 into master Oct 15, 2015
@rueckstiess rueckstiess deleted the INT-591-collection-filter branch November 12, 2015 02:04
gribnoysup pushed a commit that referenced this pull request Apr 26, 2021
* chore: Cleanup

- update dependencies
- remove  currently unused `electron-rebuild`
- remove `pre-commit` from package.json as its logic is now bundled into `mongodb-js-precommit`

* refactor: Collapse lodash dependency

Move away from `lodash.*` to top-level `lodash` that is far easier to maintain.

* chore: Allow mongodb-security semver to update automatically

* chore: connection-model update for kerberos electron 6 fix

mongodb-js/connection-model#210

* fix(test): No idea how this test ever actually passed before... ✅

* feat(ci): Default test is now electron renderer

So we can catch mismatched behaviors in the future as NODE-2387 uncovered

* chore: update mongodb-runner mongodb-js/runner#158

* docs: Add note to README that tests now run via electron-mocha
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