Move Named Layers addon to release #41317

Closed
opened 2014-08-04 17:00:20 +02:00 by Brendon Murphy · 6 comments
Member

Hi, I would like to move the
https://developer.blender.org/diffusion/BAC/browse/master/ui_layer_manager.py into release.
wiki: http://wiki.blender.org/index.php/Extensions:2.6/Py/Scripts/3D_interaction/layer_manager
The layer manager has been in contrib for too long & is a much requested feature. Given the amount of users who can immediately benefit from this & ongoing support from the author. I see no reason not to add. This addon fills a missing gap for users coming from other softwares & provides a very usable ui.
+1 from JonathanW & JTA on irc.
needs code review.

Hi, I would like to move the https://developer.blender.org/diffusion/BAC/browse/master/ui_layer_manager.py into release. wiki: http://wiki.blender.org/index.php/Extensions:2.6/Py/Scripts/3D_interaction/layer_manager The layer manager has been in contrib for too long & is a much requested feature. Given the amount of users who can immediately benefit from this & ongoing support from the author. I see no reason not to add. This addon fills a missing gap for users coming from other softwares & provides a very usable ui. +1 from JonathanW & JTA on irc. needs code review.
Author
Member

Changed status to: 'Open'

Changed status to: 'Open'
Author
Member

Added subscriber: @BrendonMurphy

Added subscriber: @BrendonMurphy
Bastien Montagne self-assigned this 2014-08-04 20:22:50 +02:00

Started review, needs heavy style and naming cleanup, quite a bit of simplification (using comprehensions e.g.), and a change in how it stores data in scene (use props group, instead of wild creation of tens of props in poor scene).

Will handle that in coming days.

Started review, needs heavy style and naming cleanup, quite a bit of simplification (using comprehensions e.g.), and a change in how it stores data in scene (use props group, instead of wild creation of tens of props in poor scene). Will handle that in coming days.

So, ended up to be nearly a complete rewrite (not logic-wise, but mostly style-wise, though there were a few minors issues fixed on the way too)…

Hope the org author won't be too much upset… :/ At first I thought it would be simple and done in one or two hours, but spent way too much time on it, next time won't do same mistake!

Anyway, thinks it's OK for main dir now, will wait a few days and if no issue shows up will do the move.

So, ended up to be nearly a complete rewrite (not logic-wise, but mostly style-wise, though there were a few minors issues fixed on the way too)… Hope the org author won't be too much upset… :/ At first I thought it would be simple and done in one or two hours, but spent way too much time on it, next time won't do same mistake! Anyway, thinks it's OK for main dir now, will wait a few days and if no issue shows up will do the move.

Changed status from 'Open' to: 'Resolved'

Changed status from 'Open' to: 'Resolved'
Done (026c9fe1c9, blender/blender-addons-contrib@6ec1c46f1a).
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender-addons#41317
No description provided.