Review of "Dhruva" version 7

Details Page Preview

A clean and customizable dock for GNOME Shell. Features: • Hover zoom with six motion styles • Panel mode that brings the top bar into the dock • Window previews with peek • App folders with emoji or custom icons • Click and minimize animations • Blur, themes, running indicators and notification badges • Auto-hide that stays out of the way of games and fullscreen apps • Music Pill with synced lyrics • Clock, Recycle Bin, folder and drive modules • Any screen edge, any monitor The Music Pill fetches lyrics from lrclib.net.

Extension Homepage
https://github.com/narkagni/dhruva

No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

Shexli (experimental)

Shexli did not find any issues.

All Versions

Version Status
5.1.5 (33) Active
5.1.4 (32) Inactive
5.1.3 (31) Inactive
5.1.2 (30) Inactive
5.1.1 (29) Inactive
5.1 (28) Inactive
5.0 (27) Inactive
4.9 (26) Inactive
4.9 (25) Rejected
4.7 (24) Inactive
4.6 (23) Rejected
4.3 (22) Rejected
4.3 (21) Rejected
4.3 (20) Rejected
3.1 (19) Rejected
3.1 (18) Rejected
2.0 (17) Inactive
1.5 (16) Inactive
1.4 (15) Inactive
1.2 (14) Inactive
1.2 (13) Inactive
1.1 (12) Inactive
11 Inactive
10 Inactive
9 Rejected
8 Rejected
7 Rejected
6 Rejected
5 Rejected
4 Inactive
3 Inactive
2 Rejected
1 Rejected

Previous Reviews on this Version

JustPerfection rejected
1. You shouldn't remove the user's keybinding (line 70-81 `extension.js`). 2. Please use `connectObject()` and `disconnectObject()` so it is easier to track for cleanup (`extension.js` process only). 3. Remove line 31, 46-49 `prefs.js`. 4. `this._isDestroyed` is a bad practice. Please remove that. 5. It's weird that `src/core/TimerRegistry.js` is handling signal connections. 6. Timeout should be removed on destroy/disable and before creating a new one: - line 82, 109 `src/core/autohide/AutoHideAnimations.js` - line 51, 70, 234 `src/core/autohide/EdgeDetection.js` - line 309 `src/ui/dock/DockItemBuilder.js` - line 109 `src/ui/dock/DockRenderer.js` - line 363, 530, 604, 874 `src/ui/magnifier/Magnifier.js` - line 265 `src/ui/magnifier/MagnifierTooltipRenderer.js` - line 37, 45, 54, 100 `src/ui/modules/DesktopButtonModule.js` - line 81 `src/ui/modules/DockModules.js` [EGO Review Guidelines: Timeout](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources) 7. Don't need try-catch wrapper (line 105 `src/core/autohide/AutoHideAnimations.js`). 8. Not clear where that cleanup method is getting called (line 406 `src/ui/context-menu/AppContextMenu.js`). 9. Cannot trace where the destroy callback is getting called (line 294 `src/ui/folder-menu/FolderMenuBuilder.js`). When you make a function that large with so many functions in it, that is not just hard to read, that's a class wrapped into a function. 10. Why are you checking whether that instance has `disconnect`? - line 1042-1065 `src/ui/magnifier/Magnifier.js` 11. How can I know on disable the `stopDragLoop` is getting called (line 38 `src/ui/magnifier/MagnifierDragLoop.js`)?