Skip to content

[4.x] List Filters#37424

Merged
laoneo merged 3 commits intojoomla:4.1-devfrom
brianteeman:patch-13
Apr 23, 2022
Merged

[4.x] List Filters#37424
laoneo merged 3 commits intojoomla:4.1-devfrom
brianteeman:patch-13

Conversation

@brianteeman
Copy link
Copy Markdown
Contributor

Steps to reproduce the issue

Create a menu of type Articles Category List
Create a menu of type List all contacts in a category

Enable the display filter option in both

Expected result

the filter field works/displays the same in both

Actual result Com_content

In com_content there is an option to select what the filter field should be and depending on the filter field the display & functionality in the front end is adjusted.

image

Filter - Author (input)

image

Filter - Title (input)

image

Filter - Month(select)

image

Actual result com_contact

In com_contact the option is just show/hide
image

image

The problem is

  1. com_contacts the form option in the admin is filter
  2. com_contacts the button in the site is filter
  3. the placeholder text (and aria-label) is search in title in com_contacts but just title filer in com_contact

For a consistent ui this PR changes search in title to title filter

Result

image

### Steps to reproduce the issue
Create a menu of type Articles Category List
Create a menu of type List all contacts in a category

Enable the display filter option in both


### Expected result
the filter field works/displays the same in both


### Actual result Com_content
In com_content there is an option to select what the filter field should be and depending on the filter field the display & functionality in the front end is adjusted.

![image](https://user-images.githubusercontent.com/1296369/160798796-47a9c09e-2f87-48f8-b3f9-a8037a4da032.png)

#### Filter - Author (input)
![image](https://user-images.githubusercontent.com/1296369/160798237-801ff1b7-5b32-46ae-b9fc-c95e8d3af541.png)

#### Filter - Title (input)
![image](https://user-images.githubusercontent.com/1296369/160798366-ff28334a-b471-439e-8506-3b2188305caf.png)

#### Filter - Month(select)
![image](https://user-images.githubusercontent.com/1296369/160798623-f39426d4-92dc-4e47-b89d-be4f037ca8b6.png)

### Actual result com_contact
In com_contact the option is just show/hide
![image](https://user-images.githubusercontent.com/1296369/160799241-017d2fa9-e763-4d2d-ba3c-9043adc3490c.png)

![image](https://user-images.githubusercontent.com/1296369/160800153-c505a4b3-f74e-47e3-aee5-e950ded3943b.png)

### Additional comments
The _problem_ is 
1. com_contacts the form option in the admin is **filter**
2. com_contacts the button in the site is **filter**
3. the placeholder text (and aria-label)  is **search in title** in com_contacts but just **title filer** in com_contact

For a consistent ui this PR changes search in title to title filter

## Result
![image](https://user-images.githubusercontent.com/1296369/160802562-2988b5d9-b35d-442f-8551-5345c46aeb3b.png)
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-4.1-dev labels Mar 30, 2022
@Quy
Copy link
Copy Markdown
Contributor

Quy commented Apr 3, 2022

I have tested this item ✅ successfully on 17c0452


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

@richard67 richard67 added the PBF Pizza, Bugs and Fun label Apr 22, 2022
@kiki-G
Copy link
Copy Markdown

kiki-G commented Apr 22, 2022

I have not tested this item.


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

@kiki-G
Copy link
Copy Markdown

kiki-G commented Apr 22, 2022

I have tested this item 🔴 unsuccessfully on 0dba586


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

@kiki-G
Copy link
Copy Markdown

kiki-G commented Apr 22, 2022

I have tested this item ✅ successfully on 0dba586


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

3 similar comments
@patvb2003
Copy link
Copy Markdown

I have tested this item ✅ successfully on 0dba586


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

@kiki-G
Copy link
Copy Markdown

kiki-G commented Apr 22, 2022

I have tested this item ✅ successfully on 0dba586


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

@kiki-G
Copy link
Copy Markdown

kiki-G commented Apr 22, 2022

I have tested this item ✅ successfully on 0dba586


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

@RickR2H
Copy link
Copy Markdown
Member

RickR2H commented Apr 22, 2022

RTC


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

@joomla-cms-bot joomla-cms-bot added RTC This Pull Request is Ready To Commit and removed PBF Pizza, Bugs and Fun labels Apr 22, 2022
@laoneo laoneo merged commit b2134d3 into joomla:4.1-dev Apr 23, 2022
@laoneo
Copy link
Copy Markdown
Member

laoneo commented Apr 23, 2022

Thanks!

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Apr 23, 2022
@laoneo laoneo added this to the Joomla 4.1.3 milestone Apr 23, 2022
Kostelano added a commit to JPathRu/localisation that referenced this pull request May 2, 2022
joomla/joomla-cms#37115 +
joomla/joomla-cms#37286 + (отдельно в 857dcac)
joomla/joomla-cms#37464 +
joomla/joomla-cms#36250 +
joomla/joomla-cms#37527 +
joomla/joomla-cms#37535 - (только для en-GB)
joomla/joomla-cms#37559 +
joomla/joomla-cms#37594 - (только для en-GB)
joomla/joomla-cms#37588 +
joomla/joomla-cms#37424 - (только для en-GB, у нас все в одном формате с другими расширениями)
joomla/joomla-cms#37475 - (только для en-GB, у нас давно исправлено)
joomla/joomla-cms#37564 +
joomla/joomla-cms#37641 - (только для en-GB)
joomla/joomla-cms#37657 +
joomla/joomla-cms#37683 +
joomla/joomla-cms#37666 +
joomla/joomla-cms#37704 +
joomla/joomla-cms#37689 +
joomla/joomla-cms#37519 +
@brianteeman brianteeman deleted the patch-13 branch July 12, 2022 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language Change This is for Translators

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants