CefView / QCefView

A Qt Widget encapsulated CEF view based on QWidget
https://cefview.github.io/QCefView/
GNU Lesser General Public License v2.1
521 stars 137 forks source link

Add Drag and drop handler #287

Closed L-Super closed 1 year ago

tishion commented 1 year ago

@L-Super 多谢你的PR,我看了下应该只有两个问题

  1. Core的PR我已经合并了,所以你可以更新一下这个PR,把Core的submodule更新到你提交的那个版本
  2. 代码格式化似乎不符合仓库里面的clang-format文件的定义,你可能需要配置一下你本地的clang-format工具,然后格式化一下修改过的代码

其他就没有问题了,感谢!

L-Super commented 1 year ago

@L-Super 多谢你的PR,我看了下应该只有两个问题

  1. Core的PR我已经合并了,所以你可以更新一下这个PR,把Core的submodule更新到你提交的那个版本
  2. 代码格式化似乎不符合仓库里面的clang-format文件的定义,你可能需要配置一下你本地的clang-format工具,然后格式化一下修改过的代码

其他就没有问题了,感谢!

OK,代码规范的问题,应该是vs code配置的保存自动格式化未按照clang-format文件进行

L-Super commented 1 year ago

@L-Super 多谢你的PR,我看了下应该只有两个问题

  1. Core的PR我已经合并了,所以你可以更新一下这个PR,把Core的submodule更新到你提交的那个版本
  2. 代码格式化似乎不符合仓库里面的clang-format文件的定义,你可能需要配置一下你本地的clang-format工具,然后格式化一下修改过的代码

其他就没有问题了,感谢!

我使用了VS Code和Clion分别进行代码格式化,似乎是正确的。我又尝试clang-format工具进行格式化:

 C:\Users\LMR\.vscode\extensions\ms-vscode.cpptools-1.16.3-win32-x64\LLVM\bin\clang-format.exe -style=file -i .\include\QCefView.h

结果一致,代码规范应该没问题