| Summary: | Add contextual UNO commands for name ranges and data validity to context menu | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | Thomas Lendo <thomas.lendo> |
| Component: | Calc | Assignee: | Gülşah Köse <gulsah.1004> |
| Status: | VERIFIED FIXED | ||
| Severity: | enhancement | CC: | erack, gulsah.1004, heiko.tietze, markus.mohrhard, philipz85, timur |
| Priority: | medium | ||
| Version: | 5.3.4.2 release | ||
| Hardware: | All | ||
| OS: | All | ||
| See Also: | https://bugs.documentfoundation.org/show_bug.cgi?id=87357 | ||
| Whiteboard: | target:6.1.0 | ||
| Crash report or crash signature: | Regression By: | ||
| Bug Depends on: | |||
| Bug Blocks: | 86349, 86899, 108988, 109178 | ||
| Attachments: | 3 example features that should be acessible in the context menu | ||
|
Description
Thomas Lendo
2017-07-06 12:28:22 UTC
Created attachment 134516 [details]
3 example features that should be acessible in the context menu
Sounds like a very helpful enhancement. But is such a context dependent function feasible, Timur, Eike, Moggi? (In reply to Thomas Lendo from comment #0) > 3. Cell C2 contains conditional formatting. If clicking at the cell, then it > should be possible to open the "Format > Conditional Formatting > > Condition..." dialog with the context menu. Already suggested in bug 87357. The 3 examples of features I mentioned in my initial post stand for all possible features that could be promoted to the user. I don't know more so it's to others to bring them up - here or in another bug(s). Ist it ok to have both features (validity and name ranges) in this bug? I hear the devs rant in my thoughts. :) And for the record, there are some use cases (collaboration with others, forgotten use of a feature in a cell, short mouse ways, overview what is applied to the selected cell ...) where a context menu entry would help to find and to (re)use a feature that is applied to a cell or cell range. https://gerrit.libreoffice.org/#/c/47925/ solves only named ranges part. The other two patches are on the way. (In reply to Gülşah Köse from comment #5) > https://gerrit.libreoffice.org/#/c/47925/ solves only named ranges part. The > other two patches are on the way. Happy to review when all is complete. Thanks Heiko, i will send seperate patches for validation and conditional format issues. fyi. @Heiko completed https://gerrit.libreoffice.org/#/c/48067/ for validity https://gerrit.libreoffice.org/#/c/48417/ for conditional format https://gerrit.libreoffice.org/#/c/47925/ for named range Gulsah Kose committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=844c92793c1399e2f14f671f964ea35e1bce24ac [Pardus] tdf#108989 Add contextual validity command. It will be available in 6.1.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Gulsah Kose committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=a626e249033fb9a7ac117e28f3152cedf98b61fc [Pardus] tdf#108989 Add contextual manage names command. It will be available in 6.1.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. (In reply to Yousuf Philips (jay) from comment #3) > (In reply to Thomas Lendo from comment #0) > > 3. Cell C2 contains conditional formatting. If clicking at the cell, then it > > should be possible to open the "Format > Conditional Formatting > > > Condition..." dialog with the context menu. > > Already suggested in bug 87357. Because of request 3 has seperate bug report, reported by Yousuf, and request 1(named ranges) and 2(validity) is completed, i'm moving that bug as RESOLVED FIXED. Feel free to change it if there is a problem. Request 3 is solved by https://gerrit.libreoffice.org/#/c/48417/ waiting for code review. Useful. Deserves to be put in https://wiki.documentfoundation.org/ReleaseNotes/6.1. (In reply to Timur from comment #12) > Useful. > Deserves to be put in https://wiki.documentfoundation.org/ReleaseNotes/6.1. Done :) Thank you all for supporting. Eike Rathke committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=707e0f4e6b6e36432c0f04f22719428a7e1a50ee coverity#1428613 check IsValidReference() return value, tdf#108989 follow-up It will be available in 6.1.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. |