Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Copy template modal missing tip #10071

Merged
merged 1 commit into from Apr 25, 2016
Merged

Conversation

infograf768
Copy link
Member

Edit a template (not a template style). Click on Copy Template toolbar button. No tip.
Patch.
Result should be:
screen shot 2016-04-24 at 10 25 50

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-staging labels Apr 24, 2016
@brianteeman
Copy link
Contributor

I have tested this item ✅ successfully on 735666f


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10071.

1 similar comment
@alikon
Copy link
Contributor

alikon commented Apr 24, 2016

I have tested this item ✅ successfully on 735666f


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10071.

@brianteeman
Copy link
Contributor

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10071.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Apr 24, 2016
@brianteeman brianteeman added this to the Joomla 3.5.2 milestone Apr 24, 2016
?>
<div id="template-manager-css" class="form-horizontal">
<div class="control-group">
<label for="new_name" class="control-label hasTooltip" title="<?php echo JHtml::tooltipText('COM_TEMPLATES_TEMPLATE_NEW_NAME_DESC'); ?>"><?php echo JText::_('COM_TEMPLATES_TEMPLATE_NEW_NAME_LABEL')?></label>
<div class="control-group span6">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why the span 6 ?

@cyrezdev
Copy link
Contributor

@infograf768 The span6 gives trouble for the label, not vertically aligned with the input.
I have commented your code, as with a simple div control-label to include the label, it solves this display issue ;-)

@infograf768
Copy link
Member Author

I added the span6 because of the position of the tip.
If I follow you (as far as I understand) and change to:

<div id="template-manager-css" class="form-horizontal">
    <div class="control-label">
        <label for="new_name" class="modalTooltip" title="<?php echo JHtml::_('tooltipText', 'COM_TEMPLATES_TEMPLATE_NEW_NAME_LABEL', 'COM_TEMPLATES_TEMPLATE_NEW_NAME_DESC'); ?>">
            <?php echo JText::_('COM_TEMPLATES_TEMPLATE_NEW_NAME_LABEL')?>
        </label>
    </div>
    <div class="controls">
        <input class="input-xlarge" type="text" id="new_name" name="new_name"  />
    </div>
</div>

then I get:

screen shot 2016-04-25 at 08 34 41

@infograf768
Copy link
Member Author

infograf768 commented Apr 25, 2016

BTW, the tip here is now displayed in the same position as in Batch modals, where it does so grace to the span6.

@rdeutz rdeutz merged commit ec5e711 into joomla:staging Apr 25, 2016
@cyrezdev
Copy link
Contributor

cyrezdev commented Apr 25, 2016

@infograf768 ok i understand your intention ;-)
But my proposal was not exactly has you supposed, but using bootstrap standard form field structure like this :

<div id="template-manager-css" class="form-horizontal">
    <div class="control-group">
        <div class="control-label">
            <label for="new_name" class="modalTooltip" title="<?php echo JHtml::_('tooltipText', 'COM_TEMPLATES_TEMPLATE_NEW_NAME_LABEL', 'COM_TEMPLATES_TEMPLATE_NEW_NAME_DESC'); ?>">
                <?php echo JText::_('COM_TEMPLATES_TEMPLATE_NEW_NAME_LABEL')?>
            </label>
        </div>
        <div class="controls">
            <input class="input-xlarge" type="text" id="new_name" name="new_name"  />
        </div>
    </div>
 </div>

capture d ecran 2016-04-25 a 11 46 11

If you want same display as inside batch (which is loaded field from layout and not constructing it as in copy modal) so you should not use form-horizontal.

There the code with exactly the batch modal rendering :

<div id="template-manager-css" class="row-fluid">
    <div class="control-group span6">
        <div class="controls">
            <label for="new_name" class="control-label modalTooltip" title="<?php echo JHtml::tooltipText('COM_TEMPLATES_TEMPLATE_NEW_NAME_DESC'); ?>"><?php echo JText::_('COM_TEMPLATES_TEMPLATE_NEW_NAME_LABEL')?></label>
            <input class="input-xlarge" type="text" id="new_name" name="new_name"  />
        </div>
    </div>
</div>

capture d ecran 2016-04-25 a 11 43 44
But the good BS2 way seems using fieldset for this display: http://getbootstrap.com/2.3.2/base-css.html#forms

@infograf768 as your PR is already merged, no need to revert it, as i've planned to change name of modal id (which should be unique) from collapseModal to copyModal and so be logical with other modals of this view (renameModal, fileModal, folderModal...) so i could do the change in the same time! ;-)

So, what's best ?

  • Batch look like (with span6)
  • Or Bootstrap form field look like (as for all edit forms)

Note: my screenshots are on a big screen, so that's why very large modals... and i can do an adjustement in the same time of the PR with new file names ;-)

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Apr 25, 2016
@infograf768
Copy link
Member Author

@JoomliC
Cyril, I honestly don't care, as long as tips are readable and interface is consistent. 😃

@cyrezdev
Copy link
Contributor

cyrezdev commented Apr 25, 2016

@infograf768 👍
I will check it to improve it, and in the same time, display hidden tooltip in some othe modals of this view (the more i work on modals, the more i see minor issues, so good to fix them all... but consistency is not easy as some modals loading iframe, some layouts for fields, and some (as here) built in view fields...)
Will inform there when PR will be ready ;-)

@cyrezdev
Copy link
Contributor

@infograf768 just created the PR #10100 with a few improvement not only for tooltip, but too for usuability of file and folder modals ;-)

Kubik-Rubik pushed a commit that referenced this pull request Apr 29, 2016
)

* Rename modal ID to significant name copyModal

* Improve com_templates template manager modals
@rdeutz rdeutz modified the milestones: Joomla 3.5.2, Joomla! 3.6.0 May 1, 2016
@infograf768 infograf768 deleted the template_modals branch May 11, 2016 06:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Language Change This is for Translators
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants