-
Notifications
You must be signed in to change notification settings - Fork 16
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
Admin screens do not overflow with Opti 12.22.0 #95
Comments
Are you waiting to see if Optimizely fix the overflow on their menu changes themselves or will you be releasing a fix for this? I could raise a PR if required. |
@marisks Are you able to review this? |
Can you make a PR? |
I have raised a support ticket with Opti around them breaking the interface, but they don't consider it an issue their end, and it's for 3rd parties to fix. I am not sure that I agree with this sentiment, but TBH I didn't have the energy to get into an argument. I am happy to make a hotfix and PR for this, as I would like to get a fix out. Give me a few days |
Maybe something to bring up in the slack optimizely slack channel |
That's ok if they change UI. But then they need documentation on how to integrate their UI with 3rd party UIs. Our current solution is also just found by trial and error. Maybe they have documentation now though, I haven't checked it lately. |
Maybe an interim fix for now? I do get where you are coming from. I changed the menus on another plug-in because of top top top menus so having this issue of left left menus is equally frustrating. I do wish they had better documentation for plugging into their menus and personally I do consider this a breaking change by Optimizely, which seems to be a bit of pattern right now. I'm happy to add my support to whatever is raised in the slack channels. |
Just wanted to comment on your discussion here. We changed the menu system in adapting to a common UI to make all Optimizely products look and behave more the same. Our goal is always not to introduce breaking changes. I personally tried many new addons with the menu system and we fixed many issues. However, Not found handler was tested but not with enough data to require the scroll, and that was in retrospect something we should have done. Geta tags, Sql Studio and others were also among the list that were tried. Yesterday I ran tests with Not found handler and could replicate the scrolling issue. We have registered a bug for it and we have a possible fix that we will run through QA to verify it. You can track the bug here if you wish. CMS-29327 Regarding if changing menus is a breaking change, I dont personally think so. But that can always be debated and all views are valid. We managed to get many addons to work without any change required. This addon should work once we fix the bug. Also the new menu system allows for two levels of menus on the left hand side. Take a look at admin for example. It has two levels. Our reasoning is that an addon that has its own menu can continue to use it. Sure it wont look like the admin menu. But it will work. If a addon wants to use our menu system instead, it can be accomplished with the IMenuProvider and adding subitems to the main item. It will load the second level once the first level is clicked upon. See example below. We know that we have been dropping some large issues lately and we apologize for that. Please know that we take it all very seriously and will continue to try and improve as we move forward. |
@DeepRed I had a look at the ticket, I had the same issue with one of my modules in terms of the menu and that does not use an iFrame. The menu change fixed the height of the main div container and I had to apply an overflow to a child container instead. |
@GeekInTheNorth thanx, good to know. The fix we have looked at is changing the height of the div and changing overflow. We believe it should function for your scenario as well. I will update the description of the bug. |
For others that are wanting an interim fix, you can do this yourself. The Admin plugin for the Geta NotFound Handler is a Razor Class Library, which means you can provide an alternate Razor file in your implementation. So in your web project, create a new razor file with the following path and name: The following is essentially the same razor file as the one in this package, except with some URL corrections on the menu links to handle the razor file being external to the DLL and the minor UI fix:
|
@GeekInTheNorth Could you please create a PR for this fix? |
@DeepRed Changing UI is ok, we expect it, but there is a lack of documentation about how to integrate our UIs in the editorial interface. There are a few features we need and need them documented:
|
@marisks It appears we don't need to fix this anymore thanks to Optimizely doing a more global fix to how they handle the menus: https://world.optimizely.com/documentation/Release-Notes/ReleaseNote/?releaseNoteId=CMS-29327 |
Hi,
With the latest release of Optimizely (12.22.0), the Admin screens are no longer responsive so the bottom of the page is cut off and the pagination isn't visible.
As you can see on the screenshot above the scroll bar is cut off.
The fix is very simple, for the containing div
overflow:auto
height: calc(100vh - 40px)
The text was updated successfully, but these errors were encountered: