[Freeipa-devel] [freeipa PR#158][comment] WebUI: update Patternfly and Bootstrap

2017-01-05 Thread pvoborni
  URL: https://github.com/freeipa/freeipa/pull/158
Title: #158: WebUI: update Patternfly and Bootstrap

pvoborni commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/18425dbbe7b7c311cf947074d505225b235df769
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/158#issuecomment-270705433
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#158][comment] WebUI: update Patternfly and Bootstrap

2017-01-05 Thread pvoborni
  URL: https://github.com/freeipa/freeipa/pull/158
Title: #158: WebUI: update Patternfly and Bootstrap

pvoborni commented:
"""
works for me
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/158#issuecomment-270705015
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#158][comment] WebUI: update Patternfly and Bootstrap

2016-11-14 Thread mbasti-rh
  URL: https://github.com/freeipa/freeipa/pull/158
Title: #158: WebUI: update Patternfly and Bootstrap

mbasti-rh commented:
"""
Bump for review
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/158#issuecomment-260331171
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#158][comment] WebUI: update Patternfly and Bootstrap

2016-10-18 Thread pvomacka
  URL: https://github.com/freeipa/freeipa/pull/158
Title: #158: WebUI: update Patternfly and Bootstrap

pvomacka commented:
"""
I added minimized patternfly and boostrap javascript files instead of classic 
ones. 
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/158#issuecomment-254518940
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#158][comment] WebUI: update Patternfly and Bootstrap

2016-10-18 Thread mbasti-rh
  URL: https://github.com/freeipa/freeipa/pull/158
Title: #158: WebUI: update Patternfly and Bootstrap

mbasti-rh commented:
"""
works for me
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/158#issuecomment-254466745
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#158][comment] WebUI: update Patternfly and Bootstrap

2016-10-12 Thread pvoborni
  URL: https://github.com/freeipa/freeipa/pull/158
Title: #158: WebUI: update Patternfly and Bootstrap

pvoborni commented:
"""
I don't think the patternfly package can be included in Fedora as is. It 
internally bundles several packages, some of them already packaged (jquery, 
fontawesome-fonts, OpenSans-fonts). 

Additionally tha package puts files on non-standard place. 

Some info about Fedora packaging:
* https://fedoraproject.org/wiki/Packaging:JavaScript
* https://fedoraproject.org/wiki/Packaging:Web_Assets
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/158#issuecomment-253276778
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#158][comment] WebUI: update Patternfly and Bootstrap

2016-10-12 Thread redhatrises
  URL: https://github.com/freeipa/freeipa/pull/158
Title: #158: WebUI: update Patternfly and Bootstrap

redhatrises commented:
"""
> @redhatrises Thank you for the comment and the link. I agree that it would be 
> really nice, but unfortunately there is no PatternFly package in Fedora. 

@pvomacka  you're right. I should have checked. They do exist in the Patternfly 
Copr repos: https://copr.fedorainfracloud.org/coprs/patternfly/

Not sure if those can be used or a request needs to be made to include those 
RPMs into Fedora?
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/158#issuecomment-253272562
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#158][comment] WebUI: update Patternfly and Bootstrap

2016-10-12 Thread pvomacka
  URL: https://github.com/freeipa/freeipa/pull/158
Title: #158: WebUI: update Patternfly and Bootstrap

pvomacka commented:
"""
@redhatrises  Thank you for the comment and the link. I agree that it would be 
really nice, but unfortunately there is no PatternFly package in Fedora. 

Anyway, I would be happy to do a review of a PatternFly package.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/158#issuecomment-253262288
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#158][comment] WebUI: update Patternfly and Bootstrap

2016-10-12 Thread redhatrises
  URL: https://github.com/freeipa/freeipa/pull/158
Title: #158: WebUI: update Patternfly and Bootstrap

redhatrises commented:
"""
@pvomacka should this use the patternfly RPM rather than having the code copied 
here as well? See 
https://www.redhat.com/archives/patternfly/2014-July/msg00017.html
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/158#issuecomment-253200408
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#158][comment] WebUI: update Patternfly and Bootstrap

2016-10-12 Thread redhatrises
  URL: https://github.com/freeipa/freeipa/pull/158
Title: #158: WebUI: update Patternfly and Bootstrap

redhatrises commented:
"""
@pvomacka should this use the patternfly RPM rather than having the code copied 
here as well? See 
https://www.redhat.com/archives/patternfly/2014-July/msg00017.html
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/158#issuecomment-253200408
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code