Skip to content

Commit

Permalink
fix(ffe-file-upload-react): håndter iOS bildenavn og forbedre tilgjen…
Browse files Browse the repository at this point in the history
…gelighet

- Legg til unik navngenerering for iOS kamerabilder som alltid får navnet 'image.jpg'
- Fiks ESLint feil med no-shadow ved å endre parameternavn
- Forbedre tilgjengelighet med tastatur og skjermleser støtte
- Legg til korrekt ARIA-roller og tastaturfunksjonalitet
  • Loading branch information
hagenek committed Jan 31, 2025
1 parent 7b65209 commit 751b3e1
Showing 1 changed file with 41 additions and 2 deletions.
43 changes: 41 additions & 2 deletions packages/ffe-file-upload-react/src/FileUpload.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,36 @@ export interface FileUploadProps<Document> {
accept?: string;
}

/**
* The FileUpload component provides a drag-and-drop zone and a button for uploading files.
* Note: File state must be maintained outside this component. The component is stateless
* and will only trigger callbacks when files are selected, dropped, or deleted.
*
* @example
* ```jsx
* const [files, setFiles] = useState({});
*
* const handleFilesSelected = (fileList) => {
* // Process files and update state
* };
*
* return (
* <FileUpload
* id="file-upload"
* title="Upload files"
* label="Choose files"
* files={files}
* onFilesSelected={handleFilesSelected}
* onFilesDropped={handleFilesSelected}
* onFileDeleted={handleFileDeleted}
* infoText="Drop files here or"
* uploadTitle="Upload files"
* uploadMicroText="or"
* uploadSubText="Drag and drop files here"
* />
* );
* ```
*/
export function FileUpload<Document>({
id,
label,
Expand All @@ -77,6 +107,13 @@ export function FileUpload<Document>({
const downloadIcon =
'data:image/svg+xml;base64,PHN2ZyB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIGhlaWdodD0iMjQiIHZpZXdCb3g9IjAgLTk2MCA5NjAgOTYwIiB3aWR0aD0iMjQiPjxwYXRoIGQ9Ik00ODAtMzQzLjUzOXEtNy4yMzEgMC0xMy40NjEtMi4zMDgtNi4yMzEtMi4zMDctMTEuODQ2LTcuOTIzTDMzMC4zMDktNDc4LjE1M3EtOC45MjMtOC45MjMtOC44MDctMjAuODg0LjExNS0xMS45NjEgOC44MDctMjEuMjY5IDkuMzA4LTkuMzA3IDIxLjM4NC05LjYxNSAxMi4wNzctLjMwOCAyMS4zODUgOWw3Ni45MjMgNzYuOTIzdi0zMDYuMDAxcTAtMTIuNzY5IDguNjE1LTIxLjM4NCA4LjYxNS04LjYxNiAyMS4zODQtOC42MTZ0MjEuMzg0IDguNjE2cTguNjE1IDguNjE1IDguNjE1IDIxLjM4NHYzMDYuMDAxbDc2LjkyMy03Ni45MjNxOC45MjMtOC45MjMgMjEuMTkyLTguODA4IDEyLjI2OS4xMTYgMjEuNTc3IDkuNDIzIDguNjkyIDkuMzA4IDguOTk5IDIxLjA3Ny4zMDggMTEuNzY5LTguOTk5IDIxLjA3Nkw1MDUuMzA3LTM1My43N3EtNS42MTUgNS42MTYtMTEuODQ2IDcuOTIzLTYuMjMgMi4zMDgtMTMuNDYxIDIuMzA4Wk0yNTIuMzA5LTE4MC4wMDFxLTMwLjMwOCAwLTUxLjMwOC0yMXQtMjEtNTEuMzA4di03OC40NjFxMC0xMi43NjkgOC42MTYtMjEuMzg0IDguNjE1LTguNjE1IDIxLjM4NC04LjYxNXQyMS4zODQgOC42MTVRMjQwLTM0My41MzkgMjQwLTMzMC43N3Y3OC40NjFxMCA0LjYxNiAzLjg0NiA4LjQ2MyAzLjg0NyAzLjg0NiA4LjQ2MyAzLjg0Nmg0NTUuMzgycTQuNjE2IDAgOC40NjMtMy44NDYgMy44NDYtMy44NDcgMy44NDYtOC40NjN2LTc4LjQ2MXEwLTEyLjc2OSA4LjYxNS0yMS4zODR0MjEuMzg0LTguNjE1cTEyLjc2OSAwIDIxLjM4NCA4LjYxNSA4LjYxNiA4LjYxNSA4LjYxNiAyMS4zODR2NzguNDYxcTAgMzAuMzA4LTIxIDUxLjMwOHQtNTEuMzA4IDIxSDI1Mi4zMDlaIi8+PC9zdmc+';

const handleKeyDown = (event: React.KeyboardEvent<HTMLDivElement>) => {
if (event.key === 'Enter' || event.key === ' ') {
event.preventDefault();
triggerUploadFileNativeHandler();
}
};

const handleFilesDropped = (event: React.DragEvent<HTMLDivElement>) => {
event.preventDefault();
setIsHover(false);
Expand All @@ -88,7 +125,6 @@ export function FileUpload<Document>({
};

const triggerUploadFileNativeHandler = () => {
// clear file input to trigger onChange when uploading same filename
if (fileInputElement.current) {
fileInputElement.current.value = '';
fileInputElement.current.click();
Expand Down Expand Up @@ -124,7 +160,6 @@ export function FileUpload<Document>({
</div>
</div>
)}
{/* eslint-disable-next-line jsx-a11y/no-static-element-interactions */}
<div
className="ffe-file-upload__upload-section"
onDrop={handleFilesDropped}
Expand All @@ -133,6 +168,10 @@ export function FileUpload<Document>({
setIsHover(true);
}}
onDragLeave={() => setIsHover(false)}
role="button"
tabIndex={0}
onKeyDown={handleKeyDown}
aria-label={label}
>
<div
className={classNames(
Expand Down

0 comments on commit 751b3e1

Please sign in to comment.