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

Wrong image position in blog view of template two #875

Closed
semideia708 opened this issue Nov 18, 2024 · 4 comments · Fixed by #853
Closed

Wrong image position in blog view of template two #875

semideia708 opened this issue Nov 18, 2024 · 4 comments · Fixed by #853
Labels
invalid This doesn't seem right
Milestone

Comments

@semideia708
Copy link

Describe the bug
The settings for positioning the images in the blog-options menu tab do not work for the 'featured articles' menu type. The images are always displayed at the top. Everything is ok for the menu type 'Category Blog'.

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'menu-blog options.'
  2. Click on 'image position.'
  3. Select "left"

Expected behavior
In the frontend. Choose featured articles. All articlles will shown with the image on the left side.

Screenshots
featured view
featured
category

System Informations (please complete the following information):

  • Joomla Version: [Joomla 5.2.1]
  • PHP Version [ PHP 8.2]
  • Astroid Version [Astroid 3.1.17]
  • Device: [laptop]
  • OS: [win 10]
  • Browser [firefox]

Additional context
Add any other context about the problem here.

@felixkat
Copy link

I can confirm this behavior.

From what I can see the code hasn't been implemented yet in 'featured', only in 'blog'

@sonvnn sonvnn added the invalid This doesn't seem right label Nov 22, 2024
@sonvnn sonvnn added this to the Version 3.1.8 milestone Nov 22, 2024
@sonvnn sonvnn linked a pull request Nov 22, 2024 that will close this issue
@sonvnn
Copy link
Member

sonvnn commented Nov 22, 2024

@semideia708 @felixkat I fixed this issue at d297a45
Please download v3-branch and test it.

Thanks & Best Regards,
Sonny

@semideia708
Copy link
Author

Yes, it works.

@felixkat
Copy link

I can also confirmed it works.

@semideia708 Can you close this issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants