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

Sticky scrolling doesn't work well with C++ template class/function #157750

Closed
xiezhaokai opened this issue Aug 10, 2022 · 3 comments
Closed

Sticky scrolling doesn't work well with C++ template class/function #157750

xiezhaokai opened this issue Aug 10, 2022 · 3 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug *duplicate Issue identified as a duplicate of another issue(s) editor-sticky-scroll
Milestone

Comments

@xiezhaokai
Copy link

xiezhaokai commented Aug 10, 2022

Env:
VS Code Version: 1.70.0
OS Version: Ubuntu 20.04

Steps to Reproduce:

Open a C++ file with template class/function
scroll down to the template function

what we expect is the class/function name , not the " template <typename T>" declaration .

2022-08-10 17-13-31 的屏幕截图

@alexdima
Copy link
Member

@xiezhaokai Do you have a copy-paste friendly sample that reproduces the problem? We would like to verify, but we think this might be also fixed by #157466

@alexdima alexdima added the info-needed Issue requires more information from poster label Aug 10, 2022
@xiezhaokai
Copy link
Author

xiezhaokai commented Aug 11, 2022

@alexdima
Below is the sample code for you to verify.
just copy it to an empty file and rename it with "test.h"

#include <string>

template <typename T >
class ProtocolVarField
{
public:
    typedef T internalVarType;
    size_t offsetInProtocol = 0;
    std::string description;
    std::string fieldName1;
    // several members to make the class body long enough
    // to test sticky-scrolling
    std::string fieldName2;
    std::string fieldName3;
    std::string fieldName4;
    std::string fieldName5;
    std::string fieldName6;
    std::string fieldName7;
    std::string fieldName8;
    //***
    //
};

2022-08-11 08-58-22 的屏幕截图

@aiday-mar
Copy link
Contributor

Hi, thanks for the feedback, I just verified on the new insiders version, and I have the following behavior:

sticky-scroll-type-cpp

@alexdima alexdima added bug Issue identified by VS Code Team member as probable bug and removed info-needed Issue requires more information from poster labels Aug 11, 2022
@alexdima alexdima added this to the August 2022 milestone Aug 11, 2022
@aiday-mar aiday-mar added the *duplicate Issue identified as a duplicate of another issue(s) label Aug 22, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Sep 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug *duplicate Issue identified as a duplicate of another issue(s) editor-sticky-scroll
Projects
None yet
Development

No branches or pull requests

5 participants