search

adaptlearning / adapt-contrib-boxMenu

A simple contributed menu based upon a grid system
GNU General Public License v3.0
2 stars 29 forks source link

New: Required and optional labels added (Fixes #191) #193

Open zubairslamdien opened 5 months ago

zubairslamdien commented 5 months ago

Fixes #191

New

StuartNicholls commented 5 months ago

Yes please to a mock-up. Personally, I think the required and optional text should sit above all the other text. It seems more logical there as you need to know first whether the text is 'required' reading. It would be good if it sits outside the flow of the other content (I think) as that'll give more options for placement of that text (so at the top of the item 'box' for example.

I'd also like to see some styling perhaps, as this feels like a separate thing from the rest of the content of that 'box' item.

StuartNicholls commented 5 months ago

I'll leave the mock-up for the boxmenu item to @zubair but I've added a simple mock-up of a suggested use case for discussion

mock-up-item