-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve breakpoints #362
Improve breakpoints #362
Comments
Thoughts please @eleanor-heath @kirsty-hames @StuartNicholls @guywillis |
This is already supported in the FW as there are no limitations assigning components to blocks and using the column classes overrides the |
We could do with padding out what the issues are/what we are trying to achieve as a starting point. Regarding updating the existing breakpoints, there are many opinions on these so I think it would be good to experiment with different breakpoint values in a real course/s and identify where additional breakpoints are needed. |
Yes, to complicate this issue, there is the question of how wrapping works with varying numbers of components. 3 is a particularly good example for explaining the complexity, with rows (3), (1,2), (2,1), (1,1,1) - how do we decide between a (1,2) and a (2,1) layout in the AAT config? |
In my conversations with people, at least a couple more breakpoints are required, though their actual values seem in contention and so I was going to introduce them unvalued so that we can explore that question further with a testing base. |
I would very much like to avoid long term goals and start with known issues. There are many intersecting pieces to this puzzle. We know, for example, that we need to move the breakpoints to cater for the modern device stack, and that adding a couple more would help. I'd like to get that done first. That is why I've broken it down into immediately actionable pieces and after that, a further discovery and alignment phase. |
The current implementation of columns uses classes and these handle the wrapping responsively. The application of classes itself isn't very AAT friendly and we do have the added complexity of responsive wrapping so multiple classes needed. Ideally this would be configurable rather than a class and we streamline this for ease. I haven't raised this as an issue on AAT as this needs some consideration in the FW first. |
I thought that the main thrust of this change was long term goals. From a production standpoint 'Known issues' / immediate issues and device issues are pretty minor issues in my opinion, so for me it makes it quite difficult to contribute in this context if we're not included long term / complex goals. We certainly need an extra breakpoint at the desktop end, but I have no idea if we'd need a smaller one at this stage. I guess, from a framework point-of-view some people might say.. do more e-learning on mobile devices?.. I'm just guessing here though. In my experience though, I never need anything below the smaller breakpoint. My initial thoughts are we may only need 4 breakpoints or more at the upper end, especially if we want to keep the middle ones similar to their historical values, but I think this stuff needs some investigation. |
Re. actual breakpoints, I have no issues with specific values, but I'd like to see them justified from either a production standpoint or if its a framework standpoint, then thats fine by me, just wanting some transparency and articulation on the matter. |
If Im following this correctly.. This is going to be a tough one to implement in the tool, but from a framework perspective, I'd imagine layouts at breakpoints will be up to the FED. I think auto reflows / wrapping of components will be very simple as a default? |
As in, if we make the breakpoints absolutely derivable from the config, names and values, then it's much easier to figure out which ones we need and the consequences of any change, as there will be a working implementation through which to explore any new values. The breakpoint stuff is not a long term goal in the sense that we know we need it and we know roughly what we don't know and fashioning a pr to explore is very straightforward. Please add longer term goals, for future discussion, to stage 2 of the issue description as a short summary. If you could be the first to suggest breakpoint values and a rationale that would kick off that part? |
@oliverfoster Okay, we had planned to do some exploitative stuff by hacking in to a framework instance, but if we're prepared to go this route then that makes perfect sense to me. |
Based upon the two prs attached, whatever one defines in the I have added two extra attributes to the schemas Using the prs, it is possible to create ones own This change should allow explorative stuff, without hacking and without changing the existing behaviour, just change the |
There is some value in having the breakpoints abbreviated somewhat, so I'd imagined something along the lines of xs, sm, md, lg, xl as that will help greatly with the columns fix (I'll explain that in step two). But more than 100% agree we shouldn't remove small / medium / large as that will have a big impact as you say. I had thought about ways we could do this (move forward and retain legacy names), but might it be easier to chat that bit though after I've added some rationale up top there and thought it though a bit more. If not possible, then fine, it just makes other bits a little more cumbersome. Size-wise. My thinking was just to tweak the middle values so things would still work if an old theme was dropped into a new set of values (assuming only adapt breakpoints were used of course). But the tweaks would target devices better and make the page wider to reflect wider screens on desktop. So 760 becomes say 768 for example (or 800 depending on whats important device-wise). |
In the prs, I have removed new #363 These prs can be approved and merged, so that we can more easily test any changes to breakpoint sizes. |
🎉 This issue has been resolved in version 6.35.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Subject of the issue
Adapt breakpoints are slipping further away from devices as they are in the market currently. Below is a rough plan of how to improve the situation whilst retaining backward compatibility.
Step one - Actionable
--adapt-device-[name]
(https://github.com/adaptlearning/adapt_authoring/blob/8d7573168bfa06d724aeec92cc45ae3ee20fb838/plugins/content/config/model.schema#L95-L127)
Step two - Discovery
Step three - Actionable phase 2
TBD
The text was updated successfully, but these errors were encountered: