AppFuse

Add bootstrap glyphs to buttons in forms, both generated and initial

Details

  • Type: Improvement Improvement
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.2.1
  • Fix Version/s: 2.2.1
  • Component/s: Tools - AMP
  • Labels:
    None

Description

Generated forms miss headings and are not generated following the new ui boostrap style.
Some of the existing forms do not follow the new ui bootstrap style.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
J. Garcia added a comment -

I have completed the update of bootstraps for struts2.
I attach the generated patch.
The patch was generated with:

git format-patch --binary HEAD^

and requires to apply the patch for APF-1313 before.
I've done it like this as it includes the bootstrap gif images for glyphs, as they must be in a certain directory structure so they are picked up.

It solves as well other issues, at least for Struts2:

http://issues.appfuse.org/browse/APF-1287
http://issues.appfuse.org/browse/APF-1276

and
http://issues.appfuse.org/browse/APF-1275 would be obsolete, if all frameworks are moved to bootstrap, that is.

As an extra, the focus on the first form field is back, with a jquery function.

Show
J. Garcia added a comment - I have completed the update of bootstraps for struts2. I attach the generated patch. The patch was generated with: git format-patch --binary HEAD^ and requires to apply the patch for APF-1313 before. I've done it like this as it includes the bootstrap gif images for glyphs, as they must be in a certain directory structure so they are picked up. It solves as well other issues, at least for Struts2: http://issues.appfuse.org/browse/APF-1287 http://issues.appfuse.org/browse/APF-1276 and http://issues.appfuse.org/browse/APF-1275 would be obsolete, if all frameworks are moved to bootstrap, that is. As an extra, the focus on the first form field is back, with a jquery function.
Hide
Permalink
Matt Raible added a comment -

Fixed - thanks for the patch!

Show
Matt Raible added a comment - Fixed - thanks for the patch!

People

  • Assignee:
    Matt Raible
    Reporter:
    J. Garcia
Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: