Skip to content

[4.0] Cassiopea: Fixing modals custom-select fields display#30097

Merged
Quy merged 1 commit intojoomla:4.0-devfrom
infograf768:4.0_frontend_searchtools_display
Jul 16, 2020
Merged

[4.0] Cassiopea: Fixing modals custom-select fields display#30097
Quy merged 1 commit intojoomla:4.0-devfrom
infograf768:4.0_frontend_searchtools_display

Conversation

@infograf768
Copy link
Member

Follow-up on #30087

Summary of Changes

Correct css for the menutype custom select field
Use correctly the background SVG for all custom fields

Testing Instructions

Make sure you have patched with #30087 to also test menutype field width.
Edit an article in frontend.
Select CMS Content => Menu (this will also show the correction for the menutype)
Any other xtd will also have the background corrected.

Actual result BEFORE applying this Pull Request

Screen Shot 2020-07-13 at 10 19 37

Expected result AFTER applying this Pull Request

Screen Shot 2020-07-14 at 11 56 02

@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev labels Jul 14, 2020
@richard67
Copy link
Member

I have tested this item ✅ successfully on bc38786

Hint for other testers: It needs to run npm to compile the scss.


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

@Quy
Copy link
Contributor

Quy commented Jul 14, 2020

I have tested this item ✅ successfully on bc38786


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

@Quy
Copy link
Contributor

Quy commented Jul 14, 2020

RTC


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

@Quy Quy removed NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev labels Jul 14, 2020
@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jul 14, 2020
@Quy Quy added the PR-4.0-dev label Jul 14, 2020
@Quy Quy added this to the Joomla 4.0 milestone Jul 16, 2020
@Quy Quy merged commit 398096a into joomla:4.0-dev Jul 16, 2020
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester and removed RTC This Pull Request is Ready To Commit labels Jul 16, 2020
@infograf768 infograf768 deleted the 4.0_frontend_searchtools_display branch July 17, 2020 07:17
dgrammatiko added a commit to dgrammatiko/joomla-cms that referenced this pull request Jul 21, 2020
…outs

* '4.0-dev' of github.com:joomla/joomla-cms: (612 commits)
  [4.0] Smart Search: Fixing ordering, order direction and disabled button (joomla#29474)
  [4.0] Generate routed Modal links for iframes when not on the root (joomla#30007)
  [4.0] Get menu directly in com_tags menu route helper (joomla#30039)
  Remove collapse when resizing from mobile to desktop (joomla#30132)
  [4.0] Wrap component output in `main` element to make Cassiopeia more accessible (joomla#29870)
  [4.0] Webauthn gmp warning (joomla#29731)
  [4.0] Refactor to return early, remove if depths and throw NotAllowed (joomla#29694)
  [4.0] CLI help text (joomla#29811)
  Feature/draggable typo fixes (joomla#29987)
  [4.0] Removing unnecessary workaround in finder indexer (joomla#30037)
  [4.0] Optimizing Smart Search for larger content (joomla#30008)
  [4.0] Fix js ajax for pre update checker (joomla#29980)
  [4.0] Cassiopea: Fixing modals custom-select fields display (joomla#30097)
  [4.0][com_fields] Fix draggable sorting (joomla#30094)
  [4.0] Correct incorrect @return documentation (joomla#30092)
  [4.0] Menu items modal: adding missing filters (joomla#30087)
  short to long php open tags with echo (joomla#30089)
  Use new Toolbar (joomla#30085)
  [4.0] Center status/date created headers (joomla#29249)
  [4.0] Fix Cassiopea searchtools alignment in modals (joomla#30077)
  ...

# Conflicts:
#	administrator/components/com_templates/src/View/Template/HtmlView.php
#	installation/sql/postgresql/base.sql
#	libraries/src/Application/AdministratorApplication.php
#	libraries/src/Application/SiteApplication.php
sakiss pushed a commit to sakiss/joomla-cms that referenced this pull request Oct 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NPM Resource Changed This Pull Request can't be tested by Patchtester

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments