Would you agree to a possible pull request ?

Submitting a pull request is always a good idea, as it increases the chances of improvement eventually making it into MantisBT core. All contributions are welcome and greatly appreciated.

Please make sure that your submissions adhere to our Coding Guidelines [1], if they don't your patch might be rejected.

[1] http://www.mantisbt.org/wiki/doku.php/mantisbt:coding_guidelines

both proposals seems fine,
there is also 0024590 which seem to answer the second one.
Wouldn't it be nice to have this PR merged before ?

I'm using mantisbt 2.20 and I integrated the modification of 0024590 to use his new EVENT.

And now after source code modification we can obtain a dropdown menu in the sidebar to reproduce the "plugin menu" in mantis v1.2.

My problem: I don't know how to do a correct pull request with my version.
Do I need to make my modification with the last version on GitHub ?

Thanks

I don't know how to do a correct pull request with my version.

Assuming you have a local clone of https://github.com/mantisbt/mantisbt with an up-to-date master branch, then git checkout -b mybranch master to create a local branch based on it, apply your changes and push mybranch to your github fork, then create the pull request.

If you need more help, don't hesitate to ask.

I made this pull request : https://github.com/mantisbt/mantisbt/pull/1530 Thanks

@Jeremy13560's PR never went anywhere. Yesterday @vboctor mentioned in chat that a community member has implemented an alternative approach using scrolling:

https://github.com/khushalVekariya/mantisbt/commit/494ab5d1b0761a11ec97bd9b3ceb11fc455c3b6c

Demo: https://komododecks.com/recordings/SgxgQUJQtV1DvEzUukif

Testing and reviewing the scrolling sidebar patch mentioned in my previous post. It looks OK overall, but I see several issues:

• Using non-standard CSS pseudo-element ::-webkit-scrollbar* which, while supported on most recent browsers on the market except Firefox, according to MDN:

This feature is non-standard and is not on a standards track. Do not use it on production sites facing the Web: it will not work for every user. There may also be large incompatibilities between implementations and the behavior may change in the future.

The standards-track alternative, CSS Scrollbar styling (scrollbar-color, scrollbar-width attributes) is currently not supported by Safari.

• Display does not look good with narrow Sidebar, as the scrollbar comes too close to the icons (see screenshot image.png)
• Does not work in responsive mode with lower screen widths (image-2.png)
• The patch is altering the base Ace template CSS files; the changes should be moved to ace-mantis.css

image.png (13,322 bytes)   

image.png (13,322 bytes)   

image-2.png (49,475 bytes)   

image-2.png (49,475 bytes)   

Use of the overflow: auto; attribute actually introduces a regression with the narrow (collapsed) sidebar, as the pop-up that appears to show the item's title when hovering over the icon with the mouse, is no longer visible.

title-hover-original.png (27,948 bytes)   

title-hover-original.png (27,948 bytes)   

title-hover-hidden-with-overflow.png (26,762 bytes)   

title-hover-hidden-with-overflow.png (26,762 bytes)   

I dropped a note to khushalVekariya back in September, but he never responded.

Work-in-progress branch where I tested this, with some fixes on top of the original commit:
https://github.com/dregad/mantisbt/commits/i25893-sidebar-scroll/