Skip to content
This repository was archived by the owner on Mar 1, 2023. It is now read-only.

[!!!][TASK] Rename formz namespace to fz#58

Merged
romm merged 8 commits into
developmentfrom
task/rename-formz-to-fz
Mar 14, 2017
Merged

[!!!][TASK] Rename formz namespace to fz#58
romm merged 8 commits into
developmentfrom
task/rename-formz-to-fz

Conversation

@romm

@romm romm commented Mar 14, 2017

Copy link
Copy Markdown
Owner

The decision has been taken to rename the namespace formz to fz in the contexts below.

The main reason is to improve readability: fz is far more ignorable by the eye than formz is. It also reduces the weight of the generated code.

  • JavaScript: the global namespace Formz is now accessible with Fz.

    Note that Formz is still accessible, but should be avoided.

    Example:

    Fz.Form.get(...)
  • CSS: the entire list of data attributes that looked like formz-* are transformed to fz-*. This does affect generated CSS files.

    Example:

    form[fz-value-gender="male"] {
        ...
    }
  • Fluid: the namespace formz, which was used in every template of the extension has been renamed to fz.

    Example:

    {namespace fz=Romm\Formz\ViewHelpers}
    
    <fz:field name="Email">
        ...
    </fz:field>

romm added 8 commits March 13, 2017 20:37
# Conflicts:
#	Documentation/06-IntegratorManual/ViewHelpers/Slot/RenderViewHelper.rst
#	Documentation/06-IntegratorManual/ViewHelpers/SlotViewHelper.rst
#	Documentation/Localization.fr_FR/06-IntegratorManual/ViewHelpers/Slot/RenderViewHelper.rst
#	Documentation/Localization.fr_FR/06-IntegratorManual/ViewHelpers/SlotViewHelper.rst
@romm romm changed the title Task/rename formz to fz [!!!][TASK] Rename formz namespace to fz Mar 14, 2017
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 88.409% when pulling 470650c on task/rename-formz-to-fz into f22216d on development.

@romm romm merged commit 1647035 into development Mar 14, 2017
@romm romm deleted the task/rename-formz-to-fz branch March 16, 2017 12:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants