Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add mechanism for specifying which window layout to load by default #51

Closed
bentoogood opened this issue Mar 28, 2013 · 1 comment
Closed

Add mechanism for specifying which window layout to load by default #51

bentoogood opened this issue Mar 28, 2013 · 1 comment

Comments

@bentoogood
Copy link
Contributor

Users can save custom layouts, but currently by default the 'Default' layout is loaded on startup.

Users should be able to specify one of the other available layouts to be loaded on startup.
@ghost ghost assigned johnhaddon Mar 28, 2013
@leonard7e leonard7e mentioned this issue Mar 19, 2015
Closed
lucienfostier referenced this issue in lucienfostier/gaffer Apr 4, 2017
@lucienfostier
ImageWriter: Added Expression node to control colorspace plug.
7e8ad35
lucienfostier referenced this issue in lucienfostier/gaffer Apr 7, 2017
@lucienfostier
ImageWriter: Added metadata node to inject colorspace name
18b33b1
@johnhaddon
Copy link
Member

At the same time as doing this, let's also cover this additional request :

when we save a new layout, show a list of previous stored layouts and be able to select one of them to be overwritten

In fact, I think we can just implement the "Save as Default" this way since "Default" is already one of the previous stored layouts...

johnhaddon added a commit to johnhaddon/gaffer that referenced this issue Jul 24, 2018
@johnhaddon
Layouts/LayoutMenu : Improve management of custom layouts
4ce6414 
- `Layouts.add()` now takes a `persistent` argument just as `Bookmarks.add()` does, and automatically takes care of saving persistent layouts into the startup location.
- `LayoutMenu` provides improved management of custom layouts
	- New "Save As/Default" menu item allows the default layout to be replaced
	- New "Save As/*" options allow other layouts to be replaced

Fixes GafferHQ#51
@johnhaddon johnhaddon mentioned this issue Jul 24, 2018
Merged
johnhaddon added a commit to johnhaddon/gaffer that referenced this issue Jul 25, 2018
@johnhaddon
Layouts/LayoutMenu : Improve management of custom layouts
f371882 
- LayoutMenu :
	- New "Default/..." submenu allows the default layout to be chosen
	- New "Save As/*" options allow previously saved layouts to be replaced
- Layouts :
	- Added `persistent` argument to `add()` method, mirroring the `Bookmarks.add()` API. This automatically takes care of saving persistent layouts into the startup location.
	- Added `setDefault()/getDefault()` and `createDefault()` methods to allow the management of a default layout.

Breaking Changes :

- Layouts :
	- Removed `save()` method. Use the `persistent` argument to `add()` and `setDefault()` instead.
	- Added applicationRoot argument to constructor. You should use `acquire()` instead anyway.
- LayoutMenu :
	- Removed `delete()` method.
- GUI config : Renamed standard layout from "Default" to "Standard".

Fixes GafferHQ#51
noizfactory pushed a commit to noizfactory/gaffer that referenced this issue Aug 15, 2018
@johnhaddon @noizfactory
Layouts/LayoutMenu : Improve management of custom layouts
6b9e880 
- LayoutMenu :
	- New "Default/..." submenu allows the default layout to be chosen
	- New "Save As/*" options allow previously saved layouts to be replaced
- Layouts :
	- Added `persistent` argument to `add()` method, mirroring the `Bookmarks.add()` API. This automatically takes care of saving persistent layouts into the startup location.
	- Added `setDefault()/getDefault()` and `createDefault()` methods to allow the management of a default layout.

Breaking Changes :

- Layouts :
	- Removed `save()` method. Use the `persistent` argument to `add()` and `setDefault()` instead.
	- Added applicationRoot argument to constructor. You should use `acquire()` instead anyway.
- LayoutMenu :
	- Removed `delete()` method.
- GUI config : Renamed standard layout from "Default" to "Standard".

Fixes GafferHQ#51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@johnhaddon @bentoogood