?
avatar PhilETaylor
PhilETaylor
21 Mar 2021

Steps to reproduce the issue

As well as the Cog Icon covering up part of the feedback messages (see: #32794 ) there is no padding at the top of the messages.

See actual result for example:

Expected result

Padding maybe...

Actual result

Screenshot 2021-03-21 at 22 02 06

System information (as much as possible)

Additional comments

avatar PhilETaylor PhilETaylor - open - 21 Mar 2021
avatar joomla-cms-bot joomla-cms-bot - change - 21 Mar 2021
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 21 Mar 2021
avatar PhilETaylor
PhilETaylor - comment - 21 Mar 2021

Screenshot 2021-03-21 at 22 07 23

avatar PhilETaylor
PhilETaylor - comment - 21 Mar 2021

Screenshot 2021-03-21 at 22 09 02

avatar PhilETaylor PhilETaylor - change - 21 Mar 2021
Title
[4] zero padding on top of messages at top of page
[4][Mobile] zero padding on top of messages at top of page
avatar PhilETaylor PhilETaylor - edited - 21 Mar 2021
avatar shanmukhi678
shanmukhi678 - comment - 22 Mar 2021

were is the code how you are checking the issues

avatar StarX0406
StarX0406 - comment - 22 Mar 2021

@PhilETaylor I would like to contribute to this project can you please assign this one to me.

avatar PhilETaylor
PhilETaylor - comment - 22 Mar 2021

@StarX0406 This project doesnt assign issues to contributors, but feel free to submit a PR and it will be tested by others.

avatar manakwadhwa2002
manakwadhwa2002 - comment - 23 Mar 2021

#32819 this pull request solves this issue.

avatar alikon alikon - change - 23 Mar 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-03-23 07:20:12
Closed_By alikon
avatar alikon alikon - close - 23 Mar 2021
avatar alikon
alikon - comment - 23 Mar 2021

please test #32819

avatar manakwadhwa2002
manakwadhwa2002 - comment - 23 Mar 2021

@alikon I have tested #32819, just need approvals from team members. If there is any other issue regarding it please let me know

avatar drmenzelit
drmenzelit - comment - 23 Mar 2021

@manakwadhwa2002 the process is to wait for "green lights" from the automated tests of the PR and 2 human tests (besides your own). At the moment some automated tests are failing, maybe you can have a look at the details in the PR.

avatar manakwadhwa2002
manakwadhwa2002 - comment - 23 Mar 2021

@drmenzelit, sure I'll have a look

avatar manakwadhwa2002
manakwadhwa2002 - comment - 23 Mar 2021

@drmenzelit @alikon , I am unable to understand the check fail in #32819. There is some npm error. It would be a great help if you can look at the error and help me resolve it if possible.
Thanks in advance

Add a Comment

Login with GitHub to post a comment