Skip to content
This repository has been archived by the owner on Jan 19, 2022. It is now read-only.

Add graph by URL #31

Closed
obfuscurity opened this issue Jul 26, 2012 · 2 comments
Closed

Add graph by URL #31

obfuscurity opened this issue Jul 26, 2012 · 2 comments

Comments

@obfuscurity
Copy link
Owner

Instead of requiring users to import from Saved Graphs. For example, a read-only Graphite user with a composed graph but no way to save it. Should be easy enough to add this in the existing accordion form.

@obfuscurity
Copy link
Owner Author

This will depend on some UI refactoring of the toolbar. Currently we have the following actions:

[Import Graphs] [New Dashboard] [Add to Dashboard ▼]

I think [New Dashboard] should be merged in as a menu option in the [Add to Dashboard ▼] dropdown. Likewise, this new [Add Graph URL] feature would seem to be an option under [Import Graphs], which should probably evolve into a dropdown as well. Choosing any of these would load the respective dialog in the accordion-group (which, btw, needs to be unified from the current two accordion-groups to a single one).

Proposed toolbar version:

[New Graphs ▼] [Add View to Dashboard ▼]
 Import Saved    Create New
 Paste URL       Existing 1
                 Existing 2

obfuscurity pushed a commit that referenced this issue Aug 8, 2012
@obfuscurity
Copy link
Owner Author

Do not like the way this is turning out. It just feels wrong that dropdown + link == open modal form. I think we'll go back to the original buttons, but with the context change happening inside the modal instead of the trigger button itself.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant