Skip to content

Commit

Permalink
Keyboard issues still not fixed
Browse files Browse the repository at this point in the history
  • Loading branch information
deusprogrammer committed Nov 5, 2023
1 parent 8927987 commit eb80eb0
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 20 deletions.
4 changes: 2 additions & 2 deletions src/renderer/App.css
Original file line number Diff line number Diff line change
Expand Up @@ -223,13 +223,13 @@ button.selected {
opacity: 0.5;
}

.removable,
.removeable,
.addable,
.openable {
position: relative;
}

.removable:hover::after {
.removeable:hover::after {
position: absolute;
top: 50%;
left: 50%;
Expand Down
3 changes: 2 additions & 1 deletion src/renderer/components/ClipTable.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ export default ({
videos,
collections,
collectionId,
allowCollectionFilter,
op,
opFn,
includeDelete,
Expand Down Expand Up @@ -65,7 +66,7 @@ export default ({
setSearchValue(value);
}}
/>
{!collectionId ? (
{allowCollectionFilter ? (
<>
<label>Clip Pack:</label>
<select
Expand Down
2 changes: 2 additions & 0 deletions src/renderer/routes/CollectionManager.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -297,6 +297,8 @@ export default () => {
) && video._id.startsWith('_')
)}
collections={filteredCollections}
collectionId={collectionId}
allowCollectionFilter
op="add"
opFn={(collectionId, videoId) => {
addToCollection(collectionId, videoId);
Expand Down
3 changes: 2 additions & 1 deletion src/renderer/routes/VideoList.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,8 @@ let VideoList = () => {
onDelete={(id, game) => {
deleteFile(id, game);
}}
includeDelete={true}
includeDelete
allowCollectionFilter
/>
</div>
);
Expand Down
1 change: 0 additions & 1 deletion src/renderer/routes/editor/AdvancedEditor.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -568,7 +568,6 @@ let AdvancedEditor = () => {
currentSliderPosition <=
batchClip.clip.endTime))
}
controls={true}
videoPosition={currentPosition}
subs={subs}
offset={offset}
Expand Down
15 changes: 0 additions & 15 deletions src/renderer/routes/editor/ClipCutter.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -93,15 +93,6 @@ let ClipCutter = () => {
if (stateRef.current.currentClip === null) {
return;
}
// let nextClip =
// stateRef.current.clips[
// Math.min(
// stateRef.current.clips.length - 1,
// stateRef.current.currentClip + 1
// )
// ];
// setCurrentSliderPosition(nextClip.startTime);
// setCurrentPosition(nextClip.startTime / 1000);
setCurrentClip((currentClip) =>
Math.min(stateRef.current.clips.length - 1, currentClip + 1)
);
Expand All @@ -111,12 +102,6 @@ let ClipCutter = () => {
if (stateRef.current.currentClip === null) {
return;
}
// let nextClip =
// stateRef.current.clips[
// Math.max(0, stateRef.current.currentClip - 1)
// ];
// setCurrentSliderPosition(nextClip.startTime);
// setCurrentPosition(nextClip.startTime / 1000);
setCurrentClip((currentClip) => Math.max(0, currentClip - 1));
break;
}
Expand Down

0 comments on commit eb80eb0

Please sign in to comment.