chore - cleanup sb navigator#2936
Conversation
|
Unable to perform a code review. You have run out of credits 😔 |
There was a problem hiding this comment.
Pull request overview
This PR performs cleanup work on the Section Box Navigator tool, focusing on code organization and maintainability improvements without adding new features.
Key Changes:
- Removed an unused refresh function to reduce code clutter
- Introduced a helper function
format_length_value()to centralize length formatting logic - Made touch-ups to window size configuration and positioning code
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| script.py | Main cleanup including removal of unused refresh function, addition of format_length_value() helper function for consistent length formatting throughout the code, and refinements to window positioning logic |
| SectionBoxNavigator.xaml | Window size adjustments and UI configuration touch-ups |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.25332+2254-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.25334+1646-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.25334+1705-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.25336+1232-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.25343+2308-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.25347+1751-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.25347+2146-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.25347+2208-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.25348+1520-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.25354+0912-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26006+2003-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26006+2004-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26007+1719-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26008+2103-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26008+2214-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26010+1645-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26010+2127-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26010+2154-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26017+0925-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26017+0934-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26017+1003-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26017+2108-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26017+2119-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26022+2117-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26022+2124-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26022+2138-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26023+2136-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26023+2141-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26025+1329-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26025+1418-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26030+2037-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26030+2039-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26030+2101-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26030+2136-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26030+2147-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26030+2212-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1043-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1111-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1304-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1323-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1433-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1538-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1543-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1553-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1612-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1624-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1738-wip |
|
📦 New work-in-progress (wip) builds are available for 6.0.0.26032+2005-wip |
Description
Checklist
Before submitting your pull request, ensure the following requirements are met:
pipenv run black {source_file_or_directory}Related Issues
If applicable, link the issues resolved by this pull request:
Additional Notes
Include any additional context, screenshots, or considerations for reviewers.
Thank you for contributing to pyRevit! 🎉