Skip to content

Commit

Permalink
fix: search result overflow
Browse files Browse the repository at this point in the history
add clip, and only doesn't clip it in FolderGridViewPopup.
  • Loading branch information
18202781743 committed May 10, 2024
1 parent 03f80b1 commit 0ee85e1
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 1 deletion.
2 changes: 2 additions & 0 deletions qml/FolderGridViewPopup.qml
Original file line number Diff line number Diff line change
Expand Up @@ -174,6 +174,7 @@ Popup {
padding: 10
interactive: false
focus: true
gridViewClip: false // TODO it maybe a bug for dtk, https://github.com/linuxdeepin/developer-center/issues/8468
activeGridViewFocusOnTab: folderGridViewLoader.SwipeView.isCurrentItem
itemMove: Transition { NumberAnimation { properties: "x,y"; duration: 250 } }
delegate: DelegateDropArea {
Expand All @@ -196,6 +197,7 @@ Popup {
paddingColumns: 2
interactive: false
focus: true
gridViewClip: false
activeGridViewFocusOnTab: folderGridViewLoader.SwipeView.isCurrentItem
delegate: DelegateDropArea {
width: folderGridViewContainer.cellWidth
Expand Down
3 changes: 2 additions & 1 deletion qml/GridViewContainer.qml
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ FocusScope {
property alias interactive: gridView.interactive
property alias padding: item.anchors.margins
property alias gridViewFocus: gridView.focus
property alias gridViewClip: gridView.clip
property ScrollBar vScrollBar
property bool activeGridViewFocusOnTab: false
property Transition itemMove
Expand Down Expand Up @@ -80,7 +81,7 @@ FocusScope {
ScrollBar.vertical: root.vScrollBar

anchors.fill: parent
clip: false // TODO it maybe a bug for dtk, https://github.com/linuxdeepin/developer-center/issues/8468
clip: true
highlightFollowsCurrentItem: true
keyNavigationEnabled: true
highlightMoveDuration: 150
Expand Down
1 change: 1 addition & 0 deletions qml/windowed/GridViewContainer.qml
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ FocusScope {
property alias delegate: gridView.delegate
property alias interactive: gridView.interactive
property alias gridViewFocus: gridView.focus
property alias gridViewClip: gridView.clip
property bool activeGridViewFocusOnTab: false
property int columns: 4
property int rows: Math.min(Math.ceil(count * 1.0 / columns), Helper.windowed.maxViewRows)
Expand Down

0 comments on commit 0ee85e1

Please sign in to comment.