? Failure

User tests: Successful: Unsuccessful:

avatar designbengel
designbengel
5 Sep 2016

Close Button on iOs doesn´t work

Bug in Bootstrap: Alerts are not closeable on iOs

Summary of Changes

Added cursor:pointer; to alerts.less and bootstrap_rtl.less to have the altert message be close-able on iOs too.

Testing Instructions

Please test on iOs (not Chrome emulation or something else)
Before applying the patch:
Type in wrong login credentials using Joomla! Protostar Template.
Try to dismiss the alert by tapping on the close button.

  • Nothing will happen

After applying the patch:
Login again with wrong credentials
Try to dismiss the altert
Alert should close now.

Documentation Changes Required

Sorry i deleted template.css completely and compiled over Template Manager, any better way to do that? Thanks!

image

e598eaf 12 Nov 2015 avatar n9iels CS
avatar designbengel designbengel - open - 5 Sep 2016
avatar designbengel designbengel - change - 5 Sep 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 5 Sep 2016
Category Templates (site) Front End
avatar joomla-cms-bot joomla-cms-bot - change - 5 Sep 2016
Labels Added: ?
avatar designbengel designbengel - edited - 5 Sep 2016
avatar brianteeman
brianteeman - comment - 5 Sep 2016

the best way to do it is to simply edit the less file and then from a
command line go to the build folder and type php generatecss.php

On 5 September 2016 at 15:26, designbengel notifications@github.com wrote:

Close Button on iOs doesn´t work

Bug in Bootstrap: Alerts are not closeable on iOs
Summary of Changes

Added cursor:pointer; to alerts.less and bootstrap_rtl.less to have the
altert message be close-able on iOs too.
Testing Instructions

Please test on iOs (not Chrome emulation or something else)
Before applying the patch:
Type in wrong login credentials using Joomla! Protostar Template.
Try to dismiss the alert by tapping on the close button.

  • Nothing will happen

After applying the patch:
Login again with wrong credentials
Try to dismiss the altert
Alert should close now.
Documentation Changes Required

Sorry i deleted template.css completely and compiled over Template
Manager, any better way to do that? Thanks!

[image: image]

https://cloud.githubusercontent.com/assets/828371/18250769/73c28c52-7385-11e6-8bb4-b2a4a394f5a7.png

You can view, comment on, or merge this pull request online at:

#11939
Commit Summary

  • Add cursor:pointer; to alert to make the close icon working on iOs

File Changes

Patch Links:


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#11939, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8UlEs0aIYhr2l_4m1sDTo8jRPcUMks5qnCaugaJpZM4J1HlO
.

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

avatar joomla-cms-bot joomla-cms-bot - change - 5 Sep 2016
Category Templates (site) Front End Templates (admin) Administration Templates (site) Front End
avatar designbengel
designbengel - comment - 5 Sep 2016

Thank you Brian! Done :)

avatar Minei3oat
Minei3oat - comment - 6 Sep 2016

You've forgotten to compile the template.css from Protostar

avatar bhavikTailored bhavikTailored - test_item - 6 Sep 2016 - Tested unsuccessfully
avatar bhavikTailored
bhavikTailored - comment - 6 Sep 2016

I have tested this item ? unsuccessfully on 6a75e03

- I have tested unsuccessfully on iPhone5c device.

avatar joomla-cms-bot joomla-cms-bot - change - 6 Sep 2016
Category Templates (site) Front End Templates (admin) Administration Administration Components SQL Postgresql MS SQL
avatar designbengel
designbengel - comment - 6 Sep 2016

Thank you @bhavikTailored - Sorry, went a bit lost by reverting everything - now it should work, can you check please?

avatar brianteeman
brianteeman - comment - 6 Sep 2016

Nope thats not correct you have 402 changed files ;)

avatar designbengel
designbengel - comment - 6 Sep 2016

Maybe i´ll just create a fresh new PR :-D

avatar brianteeman
brianteeman - comment - 6 Sep 2016

let me know if you get stuck or want me to do it

On 6 September 2016 at 22:02, designbengel notifications@github.com wrote:

Maybe i´ll just create a fresh new PR :-D


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#11939 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8airpX5Pkl8la94yGiVJ8irTwcviks5qndTbgaJpZM4J1HlO
.

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

avatar designbengel
designbengel - comment - 7 Sep 2016

Thank you Brian, i just made mistakes by reverting changes and was lost in Pull/Push :-D
New one is here: #11965

avatar designbengel designbengel - change - 7 Sep 2016
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2016-09-07 07:01:06
Closed_By designbengel
avatar designbengel designbengel - close - 7 Sep 2016

Add a Comment

Login with GitHub to post a comment