Closed argiepiano closed 2 years ago
PR #96. Feedback please!
@argiepiano - I'm working on some suggestions for this page.
Is it just me, or does this statement not make sense in this location.
That little phrase changes depending of the OG field you pick. It's a short explanation of what the field does. In this case it's telling you that adding a "Groups audience" field to an entity will do two things: it will make this entity a "group content" entity, and (2) it will determine to which group this "group content" is assigned.
Perhaps there is a better way to explain this?
Here is something I'm working on.
@argiepiano I've provided an new PR with some changes. My PR should include the changes in your original PR.
Feel free to edit my PR or use parts of it.
@stpaultim I think this looks good and makes the UI easier to understand. I'd like to give @olafgrabienski, @VasasA, @laryn and @schoenid an opportunity to chime in if they wish, since they've been involved in other OG issues.
OG module is compex. So the more informative pages are very useful. I like @stpaultim solution (PR #99). Some little things:
og_ui.admin.inc
I'd like to give [...] an opportunity to chime in
I like the overall direction, especially the collapsible help section. I'd love to have a closer look, but I'm not sure if I find time the next days. We'll see!
@VasasA and @olafgrabienski - Thanks for feedback. Those items should all be fixed.
Looks great!
Thanks @stpaultim - I added one small comment. Let's wait until you make that small change and @olafgrabienski can also take a look before merging. Thank you all!
Thanks for your patience, Alexjandro and Tim! @stpaultim I've added some minor comments directly in the PR.
@argiepiano - I've updated the PR to include the last changes that @olafgrabienski suggested.
Do we want to discuss the "Select one" label now or hold off?
I'm of the mind, that I would not want to this delay a beta release. I think that the sooner we get a beta release, the sooner we get more testers and more feedback. I would focus on changes right now that are blockers for a beta release.
Thank you, @VasasA, @olafgrabienski and @stpaultim!
The OG field settings UI leaves much to be desired. It's extremely confusing, as the "Add field" button appears below the vertical tabs, leading one to believe that the vertical tabs do something to the configuration, while in fact they are only informational.
I propose the following changes. From the current:
To one with better organization and titles:
I will submit a PR shortly for this. Feedback ppareciated!