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

Change spacer's tooltip from tooltip to popover #20584

Merged
merged 3 commits into from
Dec 1, 2018
Merged

Change spacer's tooltip from tooltip to popover #20584

merged 3 commits into from
Dec 1, 2018

Conversation

Quy
Copy link
Contributor

@Quy Quy commented May 26, 2018

Pull Request for Issue #19604 .

Summary of Changes

Change spacer's tooltip from tooltip to popover to match the other form fields.

Testing Instructions

Edit file \administrator\components\com_content\models\forms\article.xml
Change line 847 from:

		<field
			name="spacer1"
			type="spacer"
			hr="true"
		/>

to:

		<field
			name="spacer1"
			type="spacer"
			label="Label"
			description="This is a description"
		/>

Log in the backend
Edit an article
Under Images and Links, hover over Label

Expected result

Tooltip as popover

Actual result

Tooltip as tooltip

Documentation Changes Required

none

@infograf768
Copy link
Member

Works fine here.
before
screen shot 2018-05-28 at 07 42 28
after
screen shot 2018-05-28 at 07 40 59

Please correct JFormFieldSpacerTest.php (line 122) as Travis is not happy ;)

@carlitorweb
Copy link
Member

carlitorweb commented Jun 7, 2018

-------BEFORE--------

screenshot_20180607192551

--------AFTER--------

screenshot_20180607192658

@carlitorweb
Copy link
Member

I have tested this item ✅ successfully on bffb7dd


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

@SharkyKZ
Copy link
Contributor

RTC?

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Nov 20, 2018
@infograf768
Copy link
Member

Relaunched Travis

@infograf768
Copy link
Member

This PR is now useless. popovers have been implemented since.
Closing.

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Nov 20, 2018
@SharkyKZ
Copy link
Contributor

This PR is fine. Spacer does not have popovers in current staging.

@infograf768
Copy link
Member

reopening, but we have issues with tests in Travis.

@infograf768 infograf768 reopened this Nov 20, 2018
@SharkyKZ
Copy link
Contributor

SharkyKZ commented Nov 20, 2018

Need to update test.

$equals = '<span class="spacer"><span class="before"></span><span>' .
'<label id="spacer-lbl" class="hasTooltip" title="&lt;strong&gt;spacer&lt;/strong&gt;">spacer</label></span>' .
'<span class="after"></span></span>';

@infograf768
Copy link
Member

@Quy
Can you modify the test file?

…rTest.php

Co-Authored-By: Quy <quy@fluxbb.org>
@SharkyKZ
Copy link
Contributor

I have tested this item ✅ successfully on 40d0517


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

@SharkyKZ
Copy link
Contributor

This issue is still closed on the Issue Tracker.


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

@infograf768
Copy link
Member

infograf768 commented Nov 26, 2018

Reopened as pending on Issues Tracker and relaunched Drone.

@infograf768
Copy link
Member

I have tested this item ✅ successfully on 40d0517


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

@infograf768
Copy link
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Nov 26, 2018
@infograf768
Copy link
Member

@mbabker
I guess this can go in stable as totally B/C

@mbabker mbabker added this to the Joomla 3.9.2 milestone Dec 1, 2018
@mbabker mbabker merged commit fc98c8d into joomla:staging Dec 1, 2018
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Dec 1, 2018
@Quy Quy deleted the patch-10 branch December 1, 2018 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants