Skip to content

Inconsistent use of bootstrap classes #833

Closed

Description

@megoth

I'm working on register-form.hbs and noticing some strange use of Bootstrap classes, specifically not using form-group for group form elements. This has been done in other files.

I don't want to change more than I have in my commits, hence this issue. Should we make the views consistent, or simply make sure that this is done correctly in later work?

Activity

  1. ra100 commented on Oct 2, 2018

    @ra100

    Consistency is never bad.

  2. he8us commented on Oct 3, 2018

    @he8us
    Contributor

    Hello @megoth,

    I was working on correcting the HTML files. I corrected the markup in:

    • default-templates/server/index.html
    • default-views/account/register.hbs
    • default-views/account/register-disabled.hbs
    • default-views/account/register-form.hbs

    And I'm currently checking the content default-views/auth. I expect to be able to create a PR today or tomorrow.

  3. megoth commented on Oct 3, 2018

    @megoth
    ContributorAuthor

    @he8us Fantastic, Thank you so much! Looking forward to seeing the PR ^_^

  4. kjetilk commented on Oct 3, 2018

    @kjetilk
    Member

    That's great! Just try to make sure the patches are as small as possible, because I'm also making changes in those files, and great if we can avoid conflicts. :-)

  5. he8us commented on Oct 5, 2018

    @he8us
    Contributor

    @kjetilk I think this one can be resolved since the PR is merged

  6. megoth commented on Oct 5, 2018

    @megoth
    ContributorAuthor

    @he8us Kjetil is having a day off today, just so you know; maybe he resolves this today, but more probably it's going to be done on Monday ^_^

  7. kjetilk commented on Oct 7, 2018

    @kjetilk
    Member

    Resolved indeed! Thanks a lot for jumping in, @he8us !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions