git-sync/profilarr
1
0
Fork 0
mirror of https://github.com/Dictionarry-Hub/profilarr.git synced 2026-01-22 10:51:02 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix(profile): optimize format processing to avoid unnecessary updates

Browse Source
This commit is contained in:
Sam Chau
2025-08-14 03:42:28 +09:30
parent 737a262568
commit 4b03d706a5
1 changed files with 37 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

54
frontend/src/components/profile/scoring/FormatSettings.jsx
View File

@@ -18,7 +18,7 @@ const FormatSettings = ({ formats, onScoreChange, onFormatToggle, activeApp }) =
setGroupFilter(filter);
}, []);
const processSortedFormats = useCallback(() => {
const processSortedFormats = () => {
// Create a unique key for the current format set
const currentFormatKey = formats.map(f => f.id).sort().join(',');
const previousFormatKey = initialFormatsRef.current?.key;
@@ -86,28 +86,48 @@ const FormatSettings = ({ formats, onScoreChange, onFormatToggle, activeApp }) =
setSortedFormats(sorted);
setIsProcessing(false);
}, [formats]);
};
// Process formats asynchronously to avoid blocking the UI
// Only process formats when the profile actually changes (not on every score/toggle change)
useEffect(() => {
setIsProcessing(true);
setSortedFormats([]); // Clear previous sorted formats immediately
// Create a unique key for the current format set
const currentFormatKey = formats.map(f => f.id).sort().join(',');
const previousFormatKey = initialFormatsRef.current?.key;
// Use requestIdleCallback for better performance, fallback to setTimeout
if ('requestIdleCallback' in window) {
const id = requestIdleCallback(() => {
processSortedFormats();
}, { timeout: 100 });
// Only process if it's actually a different profile
if (!previousFormatKey || currentFormatKey !== previousFormatKey) {
setIsProcessing(true);
setSortedFormats([]); // Clear previous sorted formats immediately
return () => cancelIdleCallback(id);
// Use requestIdleCallback for better performance, fallback to setTimeout
if ('requestIdleCallback' in window) {
const id = requestIdleCallback(() => {
processSortedFormats();
}, { timeout: 100 });
return () => cancelIdleCallback(id);
} else {
const timeoutId = setTimeout(() => {
processSortedFormats();
}, 10);
return () => clearTimeout(timeoutId);
}
} else {
const timeoutId = setTimeout(() => {
processSortedFormats();
}, 10);
return () => clearTimeout(timeoutId);
// Same profile - just update the existing sorted formats with new values
setSortedFormats(prevSorted => {
if (!prevSorted.length) return formats;
// Create a map for quick lookup
const formatMap = new Map(formats.map(f => [f.id, f]));
// Update the sorted formats with new values while maintaining order
return prevSorted.map(sortedFormat =>
formatMap.get(sortedFormat.id) || sortedFormat
).filter(f => formatMap.has(f.id));
});
}
}, [formats, processSortedFormats]);
}, [formats]);
// Group formats based on selected groups
const groupedFormats = useMemo(() => {