? Success

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
18 Oct 2016

Pull Request for Issue #12406

Summary of Changes

Updated Hathor joomlaupdate default view to display the new choices

Testing Instructions

Before patch, they do not show.
After patch, the new views display correctly in sliders.

screen shot 2016-10-18 at 11 58 45

screen shot 2016-10-18 at 11 58 58

Documentation Changes Required

None

avatar infograf768 infograf768 - open - 18 Oct 2016
avatar infograf768 infograf768 - change - 18 Oct 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Oct 2016
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 18 Oct 2016
Category Administration Templates (admin)
avatar brianteeman
brianteeman - comment - 18 Oct 2016

Thanks for making this.

Do you think the login box would be better as center aligned or even left aligned. On a large screen with it right aligned it might be missed

http://i.tee.mn/pfHP.png

avatar infograf768
infograf768 - comment - 18 Oct 2016

The main problem I see is that this is not in staging...

@rdeutz
Shall we release 3.6.3 with this missing?

avatar rdeutz
rdeutz - comment - 18 Oct 2016

I am not merging something on release day, has to wait

avatar infograf768
infograf768 - comment - 18 Oct 2016

@brianteeman
I just managed here to get the features. Did not look into the login box at all.
I guess we also have to override finaliseconfirm.php

avatar zero-24
zero-24 - comment - 18 Oct 2016

We just need a center class in hathor that should do the trick 😄

.center {
    text-align: center;
}
avatar zero-24
zero-24 - comment - 18 Oct 2016

The same class is also used in the finaliseconfirm.php and the upload & install login pages.

avatar infograf768
infograf768 - comment - 18 Oct 2016

That will not work in Hathor as the sliders design is quite different.
Looking into it now.

avatar infograf768
infograf768 - comment - 19 Oct 2016

@brianteeman
@zero-24
Should work now. Please test again.
The login form will display float left on ltr sites, right on rtl.

avatar brianteeman brianteeman - change - 29 Oct 2016
Labels Added: ?
Removed: ?
avatar ggppdk
ggppdk - comment - 13 Nov 2016

I have tested this item ✅ successfully on 5867591


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

avatar ggppdk ggppdk - test_item - 13 Nov 2016 - Tested successfully
avatar brianteeman brianteeman - test_item - 14 Nov 2016 - Tested unsuccessfully
avatar brianteeman
brianteeman - comment - 14 Nov 2016

I have tested this item 🔴 unsuccessfully on 5867591

Almost but not quite. It is not possible to select all the options for writing the files


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

avatar brianteeman
brianteeman - comment - 14 Nov 2016

screenshot showing the second option obscured and the third option invisible

image

avatar ggppdk
ggppdk - comment - 14 Nov 2016

Yes this needs to be fixed, but the cropping is not specific to this view,

it will happen to any select field inside the slider in any view, if the select is near the bottom

Pane slider has overflow hidden

<div class="pane-slider content pane-down" style="... overflow: hidden; ...">

Solution, either change globally to remove the overflow hidden, but this will have undesired effects, think of any frontend 3rd party views using the slider and relying on overflow hidden ...

i think it is best to change hathor template for "fieldset.uploadForm"
and add min-height to it

avatar brianteeman
brianteeman - comment - 14 Nov 2016

it will happen to any select field inside the slider in any view, if the
select is near the bottom

examples please

On 14 November 2016 at 21:49, Georgios Papadakis notifications@github.com
wrote:

Yes this needs to be fixed, but the cropping is not specific to this view,

it will happen to any select field inside the slider in any view, if the
select is near the bottom

Pane slider has overflow hidden

Solution, either change globally to remove the overflow hidden, but this
will have undesired effects, think of any frontend 3rd party views using
the slider and relying on overflow hidden ...

i think it is best to change hathor template for "fieldset.uploadForm"
and add min-height to it

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#12454 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8W2hWwyGB7ym4mX_RSn20g8zK44Eks5q-NdQgaJpZM4KZmqp
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
https://brian.teeman.net/ http://brian.teeman.net/

avatar ggppdk
ggppdk - comment - 14 Nov 2016

it will happen to any select field inside the slider in any view, if the select is near the bottom

examples please

It was general comment, i do not have any examples now,
possibly no such views, because someone would have reported

so i think just fix this one case that we know about

avatar brianteeman
brianteeman - comment - 14 Nov 2016

thats my point. Comments like that kill changes without any justification.
Lets fix the issues we have without inventing mystical ones that might
appear if we are standing on one leg with the sun shining in the east

On 14 November 2016 at 21:56, Georgios Papadakis notifications@github.com
wrote:

it will happen to any select field inside the slider in any view, if the
select is near the bottom

examples please

It was general comment, i do not have any examples now,
possibly no such views, because someone would have reported

so i think just fix this one case that we know about

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#12454 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8Q-8VwejdrGnrp80Mnb7S03_FUm1ks5q-NkegaJpZM4KZmqp
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
https://brian.teeman.net/ http://brian.teeman.net/

avatar ggppdk
ggppdk - comment - 14 Nov 2016

Comments like that kill changes without any justification.

ok, but which changes are being killed ? this PR ?
justification to stop what ? where ?
i really do not understand which comment was stopping something, and why it was being stopped

i simply said any view that uses slider (or any similar overflow-hidden container) and has a select near the bottom has this problem, Joomla or 3rd party view

about stopping changes

i did post a successful test,
i did not post an unsuccessful test
i did say that the cropping is not related to this PR,

so if someone wants to fix an issue here, that is not related to this PR,
then sure go ahead, i even suggested 1 way to do it

so i am little confused, about killing changes


Now about testing PRs,
is it preferable that people do not post feedback, ?

  • especially feedback that explains instead of only reporting

is it preferable that people do not test the PRs ? and eventually people that created the PRs,

  • close them after frustration of not having testers ?

and about the cropping effect i had faced it before a couple of times in my own views, and it is not the first time that it is mentioned in this tracker (i do not remember where)

select2 has solved both issues of overflow-hidden and the z-index+position in parent nodes,
by moving the drop-down list directly under the BODY tag, i am using it

avatar infograf768
infograf768 - comment - 15 Nov 2016

To simplify, added min-height: 200px; to fieldset.uploadform as suggested.
We now get:
screen shot 2016-11-15 at 07 58 06

avatar brianteeman brianteeman - test_item - 15 Nov 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 15 Nov 2016

I have tested this item ✅ successfully on 74afe78


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

avatar ggppdk ggppdk - test_item - 15 Nov 2016 - Tested successfully
avatar ggppdk
ggppdk - comment - 15 Nov 2016

I have tested this item ✅ successfully on 74afe78


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

avatar brianteeman brianteeman - change - 15 Nov 2016
The description was changed
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 15 Nov 2016

RTC


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

avatar brianteeman brianteeman - change - 15 Nov 2016
Labels Added: ?
avatar brianteeman brianteeman - edited - 15 Nov 2016
avatar rdeutz rdeutz - change - 15 Nov 2016
Milestone Added:
avatar rdeutz rdeutz - change - 15 Nov 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-11-15 19:10:02
Closed_By rdeutz
avatar rdeutz rdeutz - close - 15 Nov 2016
avatar rdeutz rdeutz - merge - 15 Nov 2016
avatar rdeutz rdeutz - reference | 15c3fc4 - 15 Nov 16
avatar rdeutz rdeutz - merge - 15 Nov 2016
avatar rdeutz rdeutz - close - 15 Nov 2016

Add a Comment

Login with GitHub to post a comment