Skip to content

[4.0] Add bottom-margin in vote#33340

Merged
chmst merged 2 commits intojoomla:4.0-devfrom
rjharishabh:padding-vote
Apr 27, 2021
Merged

[4.0] Add bottom-margin in vote#33340
chmst merged 2 commits intojoomla:4.0-devfrom
rjharishabh:padding-vote

Conversation

@rjharishabh
Copy link
Copy Markdown
Contributor

@rjharishabh rjharishabh commented Apr 26, 2021

Pull Request for Issue #33337.

Summary of Changes

Add mb-2 in the voting form

Testing Instructions

  • Enable vote plugin with position-top
  • Enable vote in article
    And see the changes

Actual result BEFORE applying this Pull Request

No bottom padding applied in vote form

before-vote

Expected result AFTER applying this Pull Request

Bottom padding applied in vote form

after-vote

Documentation Changes Required

None

@rjharishabh rjharishabh changed the title [4.0] Add padding-bottom in vote [4.0] Add bottom-padding in vote Apr 26, 2021
@ghost
Copy link
Copy Markdown

ghost commented Apr 26, 2021

I have tested this item ✅ successfully on d7ee344

Thanks @rjharishabh


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

@C-Lodder
Copy link
Copy Markdown
Member

Best practise is to use margins for block separation

@rjharishabh rjharishabh changed the title [4.0] Add bottom-padding in vote [4.0] Add bottom-margin in vote Apr 26, 2021
@rjharishabh
Copy link
Copy Markdown
Contributor Author

Thanks @C-Lodder

@Quy
Copy link
Copy Markdown
Contributor

Quy commented Apr 26, 2021

I have tested this item ✅ successfully on f6b377b


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

1 similar comment
@YatharthVyas
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on f6b377b


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

@alikon
Copy link
Copy Markdown
Contributor

alikon commented Apr 27, 2021

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Apr 27, 2021
@chmst chmst added this to the Joomla 4.0 milestone Apr 27, 2021
@chmst chmst merged commit 1c161d5 into joomla:4.0-dev Apr 27, 2021
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Apr 27, 2021
@chmst
Copy link
Copy Markdown
Contributor

chmst commented Apr 27, 2021

Thanks!

@rjharishabh rjharishabh deleted the padding-vote branch April 27, 2021 08:35
@rjharishabh
Copy link
Copy Markdown
Contributor Author

Thanks everyone

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants