Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Accessible breadcrumb current folder info broken #6022

Closed
fschade opened this issue Nov 14, 2021 · 1 comment · Fixed by #6690
Closed

Accessible breadcrumb current folder info broken #6022

fschade opened this issue Nov 14, 2021 · 1 comment · Fixed by #6690
Assignees
Labels
Type:Bug Something isn't working

Comments

@fschade
Copy link
Contributor

fschade commented Nov 14, 2021

Steps to reproduce

  1. login
  2. create some folders and resources
  3. inspect <p class="... oc-breadcrumb-sr"> ...

Expected behaviour

if folder is empty and you add some content it should update (reactivity), then it should render the info with the itemCount instead of the raw output.

Actual behaviour

no reactivity and no placeholder replacement

Screenshot

Bildschirmfoto 2021-11-14 um 15 25 44

@fschade fschade added the Type:Bug Something isn't working label Nov 14, 2021
@pascalwengerter
Copy link
Contributor

Seems like it needs a $gettextInterpolate from my first understanding

@pascalwengerter pascalwengerter self-assigned this Mar 30, 2022
pascalwengerter added a commit that referenced this issue Mar 30, 2022
pascalwengerter added a commit that referenced this issue Mar 30, 2022
pascalwengerter added a commit that referenced this issue Mar 30, 2022
pascalwengerter added a commit that referenced this issue Mar 30, 2022
pascalwengerter added a commit that referenced this issue Mar 31, 2022
@kulmann kulmann moved this to Done in Web Support Board Jun 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type:Bug Something isn't working
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants