Skip to content

[4.0] Add noshadow to Article Version Modal Toolbar#33858

Merged
rdeutz merged 1 commit intojoomla:4.0-devfrom
YatharthVyas:fix-version-modal
May 14, 2021
Merged

[4.0] Add noshadow to Article Version Modal Toolbar#33858
rdeutz merged 1 commit intojoomla:4.0-devfrom
YatharthVyas:fix-version-modal

Conversation

@YatharthVyas
Copy link
Copy Markdown
Contributor

Pull Request for Issue #33848

Summary of Changes

.noshadow class added to the Toolbar

Testing Instructions

  1. Install the demo data
  2. Visit Frontend
  3. Visit any article
  4. Click on the Article's Edit icon
  5. On the edit view, click the Versions button. (Found at the bottom next to Save and Cancel)

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

image

Documentation Changes Required

None

@rjharishabh
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on c092446


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

1 similar comment
@ghost
Copy link
Copy Markdown

ghost commented May 14, 2021

I have tested this item ✅ successfully on c092446


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

@richard67
Copy link
Copy Markdown
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label May 14, 2021
@richard67 richard67 added this to the Joomla 4.0 milestone May 14, 2021
@rdeutz rdeutz merged commit e770fe3 into joomla:4.0-dev May 14, 2021
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label May 14, 2021
@YatharthVyas
Copy link
Copy Markdown
Contributor Author

Thankyou

@YatharthVyas YatharthVyas deleted the fix-version-modal branch July 9, 2021 06:50
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.

5 participants