03-09-2018, 09:09 PM,
|
|
afcouling
Senior Member
   
|
Posts: 123
Threads: 16
Joined: Aug 2015
|
|
RE: Dismissible notification boxes
I'll stick it all in a feature branch later today and share.
I would prefer to keep the notification messages within page flow, so like your suggestion.
Can you help with the Javascript?
Why would we need to switch the display attribute to inline?
Andy.
https://www.linkedin.com/in/andrewcouling
|
|
03-09-2018, 09:59 PM,
|
|
afcouling
Senior Member
   
|
Posts: 123
Threads: 16
Joined: Aug 2015
|
|
RE: Dismissible notification boxes
I'll create the feature branch firstly, then share.
(03-09-2018, 09:38 PM)falkoner Wrote: What I was suggesting was that the display attribute was set to "none" when the PHP created them, so that they weren't drawn by the browser. Then after moving them change this setting to inline (block might work as well) so they then appear in the right place.
That makes sense.
Thanks.
Andy.
https://www.linkedin.com/in/andrewcouling
|
|
03-12-2018, 10:35 AM,
|
|
afcouling
Senior Member
   
|
Posts: 123
Threads: 16
Joined: Aug 2015
|
|
RE: Dismissible notification boxes
I've managed to move the messages from the footer to the header, using appendChild().
I'm struggling with setting display correctly.
The messages appear momentarily in the footer, before moving to the header.
If I set display: none in the footer, then this style seems to carry over to the header.
Some help with this would be appreciated.
The feature branch is at https://github.com/andrewcouling/webERP/...ssible-msg
Andy.
https://www.linkedin.com/in/andrewcouling
|
|
03-12-2018, 09:46 PM,
|
|
afcouling
Senior Member
   
|
Posts: 123
Threads: 16
Joined: Aug 2015
|
|
RE: Dismissible notification boxes
Thanks Tim.
It worked fine on XAMMP with almost up-to-date repo code.
The modified files themselves were current when I branched.
I'm away from my development machine today. Will have a look this evening.
The message div at 35% width looks crap on a mobile device. I think I'll revert to full width.
Andy.
https://www.linkedin.com/in/andrewcouling
|
|
|