Success
-
Success
continuous-integration/drone/pr
the build was successful
Details
-
Success
continuous-integration/travis-ci/pr
The Travis CI build passed
Details
-
Success
continuous-integration/appveyor/pr
AppVeyor build succeeded
Details
User tests:
Successful:
Unsuccessful:
Summary of Changes
In some cases lack of set width for td element and 100% width for td with 'today' class breaks the table.
Set minimum width of td fields to 38px and removed 100% from td with 'today' class.
Testing Instructions
You can see the problem for example on Aura template from Themeforest (https://themeforest.net/item/aura-responsive-multipurpose-joomla-template/11509513), here is a screenshot from that: http://imgur.com/WOSz6FP
Expected result
All td elements in table should have the same width.
Actual result
Before the fix the td element with 'today' class tried to take as much space as possible, see here: http://imgur.com/a/T6Jgf
dmr86
-
open
-
10 May 2017
| Title |
| … |
Set minimum width of td fields |
| Set minimum width of td fields for calendar field |
|
| Title |
| … |
Set minimum width of td fields |
| Set minimum width of td fields for calendar field |
|
franz-wohlkoenig
-
change
-
10 May 2017
| Status |
Pending |
⇒ |
Discussion |
franz-wohlkoenig
-
change
-
10 May 2017
| Status |
Discussion |
⇒ |
Closed |
| Closed_Date |
0000-00-00 00:00:00 |
⇒ |
2018-03-19 23:49:41 |
| Closed_By |
|
⇒ |
Quy |
joomla-cms-bot
-
change
-
19 Mar 2018
| Closed_Date |
2018-03-19 23:49:41 |
⇒ |
2018-03-19 23:49:43 |
| Closed_By |
Quy |
⇒ |
joomla-cms-bot |
joomla-cms-bot
-
close
-
19 Mar 2018
joomla-cms-bot
-
change
-
19 Mar 2018
Add a Comment
Login with GitHub to post a comment
doesnt the rtl css file need updating as well?