Joomla-cms: [4.0] Offsets separated by white space(s) are deprecated, use a comma (,) instead.

Created on 19 Nov 2017  路  8Comments  路  Source: joomla/joomla-cms

Steps to reproduce the issue

Joomla 4 Alpha 2
Mouseover a popover e.g:

screen recording 2017-11-19 at 07 29 pm

Expected result

No console messages

Actual result

repeatedly
screen shot 2017-11-19 at 19 30 17

System information (as much as possible)

http://screenshot.myjoom.la/3w1F2K2y1f05

No Code Attached Yet

Most helpful comment

deleting an error message is not a fix for removing the error - ever

All 8 comments

Issue confirmed.


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

the error is generated because of the following line of code in bootstrap.bundle.js
console.warn("Offsets separated by white space(s) are deprecated, use a comma (,) instead.")

I don't really find any use of this line of code or any important error that this code might signify.
And removing it does solve the issue.


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

deleting an error message is not a fix for removing the error - ever

Alright. A small change in bootstrap.php at line 259 solved it.

earlier:
$opt['offset'] = isset($params['offset']) ? $params['offset'] : '0 0';
now:
$opt['offset'] = isset($params['offset']) ? $params['offset'] : '0,0';


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

Are you going to submit a pull request or do you expect someone else to do it (because thats not how it works)

I was waiting for a confirmation. Anyway, I've made a PR.


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

Set to "closed" on behalf of @franz-wohlkoenig by The JTracker Application at issues.joomla.org/joomla-cms/18724

closed as having Pull Request #19975

Was this page helpful?
0 / 5 - 0 ratings

Related issues

mbabker picture mbabker  路  4Comments

MartijnMaandag picture MartijnMaandag  路  6Comments

brianteeman picture brianteeman  路  5Comments

joomleb picture joomleb  路  3Comments

chmst picture chmst  路  5Comments