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

News app destroys Xcillion menu formatting #10

Open
TopCat6 opened this issue Feb 14, 2021 · 3 comments
Open

News app destroys Xcillion menu formatting #10

TopCat6 opened this issue Feb 14, 2021 · 3 comments

Comments

@TopCat6
Copy link

TopCat6 commented Feb 14, 2021

I'm submitting a ...

[x] bug report => search github for a similar issue before submitting

...about
[x] DNN parts

Current behavior

After adding News - Pro 3.1 (v. 03.01.04) App to a page, the page's main menu gets messed up and moved.
Expected behavior

Adding News App to page should not affect the page's menu.
Minimal reproduction of the problem with instructions

Create a page with Xcillion skin showing menu at top right. Add 2sxc 11.11.04 App module to page. Choose News app for the module (News - Pro 3.1 (v. 03.01.04)). Accept default view. Refresh the page. Page menu is now top left in a column instead of top right in a row.

Screenshot of menu using News App (problem):
image

Screenshot of menu after changing app to blu imp image gallery (normal):
image

What is the motivation / use case for changing the behavior?

Reporting apparent bug.
Please tell us about your environment:

Windows 10

  • 2sxc version(s): 11.11.04
  • Browser: [Chrome 88.0.4324.150 | Firefox 85.0.2 | Edge 88.0.705.68 ]
  • DNN: [ 9.8.1 ]
  • Language: [ English ]

Anything you would like to add

@iJungleboy
Copy link
Contributor

Hi @TopCat6 we would love to help you, but we need more information to efficiently use our (free) time.

We don't use Xcillion and there are so many Themes / Skins out there (some of them poorly implemented), that we cannot test against all or even guarantee that our initial styling is already perfect for that.

We would need to know what styles are causing this and what would need to be fixed, because it's also possible that Xcillion implements Bootstrap badly - in which case it's maybe not an issue of this app.

BTW: A very common cause for this can also be that your skin may be missing the koi.json, in which case the news App may end up loading Bootstrap-files it doesn't need to. So before doing anything else, I would check that. See also https://connect-koi.net/css-frameworks

@TopCat6
Copy link
Author

TopCat6 commented Feb 15, 2021

Thanks @iJungleboy, I was using Xcillion because it comes with DNN, so I thought it would be a good one to test with. I checked and it does have a koi.json file that says it uses bs3:

"default": {
		"cssFramework": "bs3"
	}

I suppose this could be a bug in the Xcillion skin. If so, I should raise the issue there. Any insight from you would be helpful. I did try changing to Porto skin from Mandeeps and that worked fine, but I would like to resolve the issue with the Xcillion skin. Thanks.
BTW, when it is displaying the page for "Article Not Found" the menu displays properly.

@iJungleboy
Copy link
Contributor

@TopCat6 Thanks for the feedback.

Since we don't use it, we won't prioritize this. If you can find out what rule(s) cause the problem etc. we can fix it, but we cannot invest the time to research and fix this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants