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

Padrino Admin Twitter Boostrap version #923

Merged
merged 27 commits into from Oct 19, 2012
Merged

Padrino Admin Twitter Boostrap version #923

merged 27 commits into from Oct 19, 2012

Conversation

WaYdotNET
Copy link
Contributor

Padrino Admin TwitterBootstrap 2.1.1 now :

  • Slim, Haml, Erb engine
  • Remove all locale, now only English
  • Remove all unused helper
  • Padrino-Modal: padrino-flash and delete function into modal :D
  • highlights error field
  • Automatic replace true or false with relative image (list.slim, list.haml, list.erb)
  • Breadcrumbs (Padrino::Helpers)
  • Automatic time_ago_in_words when column model is created_at or updated_at
  • Migrate function for MiniRecord
  • tag_icon helper

UPDATE:

  • Update layout
  • In Production now use compiled file (coffee & less)
  • Multiple destroy 

Retro compatibility (0.10.x)

With this version of Padrino admin, we remove some helper... but if you use old generate page, you need this plugin:

padrino g plugin https://raw.github.com/WaYdotNET/padrino-recipes/master/plugins/old_admin_plugin.rb

@travisbot
Copy link

This pull request fails (merged ae66d65 into 777aa75).

@travisbot
Copy link

This pull request fails (merged 484d124 into 777aa75).

@WaYdotNET
Copy link
Contributor Author

http://padrino.waydotnet.com/admin
user: test@padrinorb.com
password: test

@DAddYE
Copy link
Member

DAddYE commented Aug 31, 2012

Thanks man! Seems great! Label tag is a bit small, and cut the text. Bordered boxes (like login) has some like 2/3px of border, should you cut everything at 1px ?

@nesquena @achiu Im going to merge this, what do you think?

@travisbot
Copy link

This pull request fails (merged 8192277 into 777aa75).

@travisbot
Copy link

This pull request passes (merged fe31e55 into 777aa75).

@travisbot
Copy link

This pull request passes (merged 62399583 into 777aa75).

@DAddYE
Copy link
Member

DAddYE commented Sep 5, 2012

@nesquena @achiu Im ready to merge this, can I ? What do you think?

@WaYdotNET thanks for your awesome work, If you can a more cleanup and polished version (main page, readme, borders size etc...) would be great. :*

@gnepud
Copy link

gnepud commented Sep 7, 2012

🆒

@ghost
Copy link

ghost commented Sep 18, 2012

I like

@DAddYE
Copy link
Member

DAddYE commented Sep 18, 2012

@WaYdotNET pleaaaaaaaaaaaaase check always tests.

@WaYdotNET
Copy link
Contributor Author

in week :D sorry :P (add also new feature :D )

@WaYdotNET
Copy link
Contributor Author

Sorry for delay...

New function to add:

Update layout
In Production now use compiled file (coffee & less)
Multiple destroy

website demo update with new version:

http://padrino.waydotnet.com/admin

@ghost
Copy link

ghost commented Oct 13, 2012

why nobody's pulling this awesome work?!

PULL! PULL! PULL!

@antonh
Copy link

antonh commented Oct 13, 2012

Yes, please pull this to the main branch! It's stable and really looks beautiful.

@nesquena
Copy link
Member

Yes, we are looking to pull this in soon. Looks great. What do you think @DAddYE?

@DAddYE
Copy link
Member

DAddYE commented Oct 18, 2012

Yup, we need only to fix few tests unless @WaYdotNET do it by himself and the we are ready to go, sure next version will use this awesome admin!

@WaYdotNET
Copy link
Contributor Author

fix done :D

DAddYE added a commit that referenced this pull request Oct 19, 2012
Padrino Admin Twitter Boostrap version
@DAddYE DAddYE merged commit eed09dc into padrino:master Oct 19, 2012
@nesquena
Copy link
Member

Yay, great stuff guys. Thanks so much for your hard work on this.

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.

None yet

8 participants