Open rhw-repo opened 1 year ago
Preparation Branched from feature/refactor-code-smell new branch feature/refactor-css-modules, both merged into dev on completion Refactored directory structure:
Globally used components stay within components directory Components used only within one particular page moving into dedicated directory for that page, located within the pages directory. Should see all the specific components for one page within its own directory, easier to find. Also refactoring current single stylesheet index.css into global.css stylesheet + css modules
Clearer structure in anticipation of adapting to responsive design Also more intuitive structure for any future understanding, maintenance, collaboration
Bug with infinite re-render in CreateNew.js now fixed and merged into dev.
Same bugfix for infinite re-render in CreateNew.js now merged into main. Branched from main, merged dev into new branch. Manually tested. Merged new branch into main: directory structure and css updates currently consistent in main and dev. Continuing setting up css refactoring in dev.
Table refactoring underway, breakpoints change layout to grid at 1370px and limit table cell width at 480px. Currently simplified table row filtering @ first breakpoint: filter components rendered Global Filter and Date Range Filter.
Table refactoring complete, rows and pagination. Next items: Table Options Container display:grid at media breakpoints complete. Now needs display sizing of GlobalFilter adjusting at media breakpoints to prevent overflowing container.
Project pause due to vacation until 6th Nov
Pushed feature/responsive-css now please note not complete, WIP.
UPDATE 01/12/2023
Table, table's options container & table pagination completed
Footer completed
ContentDetail completed
Edit completed
WIP:
Login & Signup pages - checking rendering on certain IPad 7 iPhone screen sizes next, otherwise complete
Navbar:
problematic to display unknown length email - has been displayed as placeholder Design solution: require username at sign up, limit allowed characters to letters and numbers and a max length, display username in navbar
Logout button needs display to scale down at smaller screen sizes, too prominent for a secondary button
CreateNew needs review
Ensuring all touch targets are minimum 44 x 44 px for accessibility and usability at all screen sizes
TODO: add a wrapper around DateRangeFilter and use it to change display orientation for @media (max-width700px) UPDATE: abandoned - client requests DateRangeFilter should not be displayed at screen width less than 700px as app is intended for use as desktop first
Project designed for desktop yet should display and respond correctly across different devices Ideally the following should happen after bugfix
(for inifinte loop on render of ExperimentalAllTagsSelect in CreateNew.js and renaming of ExperimentalAllTagsSelect to a much shorter name)
Reviewing all frontend components identify where display is poor at 320px List out components & issues Table: Strategy for responsive table: adding data-title may be appropriate Generally: Width and height causing issues, review to refactor without (Flexbox) and/or review to write media queries