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.

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

Active Admin - Add multiple index pages to a resource #1745

Closed

Conversation

joshuacollins85
Copy link
Contributor

Multiple index pages can exist for a single resource so that a user can toggle between different views. The switching mechanism follows the design of scopes. Documentation has been updated.

@@ -0,0 +1,113 @@
# Arbre Components

Arbre allows the creation of shareable and extendable HTML components and is used throughout Active Admin to create view components.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please update these to a better column length. We shouldn't have to wrap to view the docs.

@pcreux
Copy link
Contributor

pcreux commented Dec 11, 2012

@joshuacollins85 Could you rebase this against master?

@macfanatic
Copy link
Contributor

This seems like a great idea, love to see it integrated.

@@ -49,7 +49,7 @@ def index(options = {}, &block)
options[:as] ||= :table
config.set_page_presenter :index, ActiveAdmin::PagePresenter.new(options, &block)
end

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see whitespace :[

@seanlinsley
Copy link
Contributor

@joshuacollins85 are you still alive? This really needs to be rebased on master.

joshuacollins85 added a commit that referenced this pull request Apr 3, 2013
Add support for multiple index page types for a resource.

Squashed commit of the following commits from
joshuacollins85/aa-index-view-change:

* 4c99fcb
* 9c77688
* e2240bf
* 5ea921c
* fd2993a
* 7f43798
* 9f3a773
* 35996b4
* bb5499c
* c2a86f5
* f5eb785
* da4ed06
* 828c1ec
* 687a3b2
* 555c329
@gregbell
Copy link
Contributor

gregbell commented Apr 3, 2013

I squashed and merged this into master in 3d722c3.

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

Successfully merging this pull request may close these issues.

5 participants