Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

Demo page looks ugly with BS3, needs update to new styling #1243

Closed
pkozlowski-opensource opened this issue Nov 9, 2013 · 13 comments
Closed

Comments

@pkozlowski-opensource
Copy link
Member

No description provided.

@lazychino
Copy link
Contributor

Hello @pkozlowski-opensource, you have something specific that you don't like or like want changed?

And I was thinking about modding a little the timepicker look on BS3
screenshot from 2013-11-21 23 49 43
I thinks it looks odd, should I work as part of this issues or as separate one?

@pkozlowski-opensource
Copy link
Member Author

@lazychino I've opened this issue for the overall design of the demo page itself. We can do a time-picker specific fix in #1285. I'm not sure how it should look for BS3 as I didn't look at the existing solutions.

@thienedits
Copy link
Contributor

What about adding a sidenav? Bootstrap 3.0 has a really wide layout. I can try to add a sidenav and fix the look of the tabs.

@gmaliar
Copy link

gmaliar commented Nov 26, 2013

Hi guys, after patching the modal ui.bootstrap today to work with bootstrap 3 I looked at what's missing in the milestones to release it, I saw the demo page was a milestone postponing release so I forked it and did some work on it.
I'm not css savvy so I didn't completely styled the colors but that's pretty easy as well so for the time being I fixed the markup to fit Bootstrap3.

#1309

@thienedits
Copy link
Contributor

I almost have something I can make a pull request with. I just need to try and add this angular affix plugin for the sidebar.

https://github.com/mgcrea/angular-bootstrap-affix

@thienedits
Copy link
Contributor

I am getting this weird git commit error:

git commit -m "update ui"
INVALID COMMIT MSG: does not match "type scope: subject" ! was: "update ui"
Note: scope must be only letters.

@gmaliar
Copy link

gmaliar commented Nov 27, 2013

A proper commit message in angular-ui/bootstrap is 

git commit -m “chore(demopage): Updated UI”
git commit -m “bugfix(carousel): Change kittens to puppies”
etc. etc.

it’s the type (chore/bugfix etc.) the scope of the work (carousel/accordion etc.) and the message.

-- 
Guy Maliar
Sent with Airmail

On November 27, 2013 at 10:41:03 AM, thienedits (notifications@github.com) wrote:

I am getting this weird git commit error:

git commit -m "update ui"

INVALID COMMIT MSG: does not match "type scope: subject" ! was: "update ui"
Note: scope must be only letters.


Reply to this email directly or view it on GitHub.

@thienedits
Copy link
Contributor

awesome thanks that worked

@kristianmandrup
Copy link

So how does it look now? ;)

@Mitdasein
Copy link

It looks like i wasted two hours trying to get Dropdown Toggle to work, is what it looks like.

@thienedits
Copy link
Contributor

I upload some PNG screenshots. Download the files and view them at "Actual/Full Size".

https://drive.google.com/folderview?id=0B0qdt7xwL6A8cldMZS1odjhHR0k&usp=sharing

pkozlowski-opensource pushed a commit that referenced this issue Dec 27, 2013
Adds new header, mobile menu, fastclick, smooth scrolling, and more.

Closes #1243
Closes #1469
@pkozlowski-opensource
Copy link
Member Author

Fixes are in the bootstrap3 branch now!

@thienedits
Copy link
Contributor

nice 👍

bekos pushed a commit that referenced this issue Dec 28, 2013
Adds new header, mobile menu, fastclick, smooth scrolling, and more.

Closes #1243
Closes #1469
pkozlowski-opensource pushed a commit that referenced this issue Dec 28, 2013
Adds new header, mobile menu, fastclick, smooth scrolling, and more.

Closes #1243
Closes #1469
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

6 participants