uzh / marugoto

Marugoto: eLearning Framework for Story Telling
GNU General Public License v2.0
8 stars 6 forks source link

Minor thread 🤘🗒️ #90

Closed interrogator closed 4 years ago

interrogator commented 5 years ago

Collection of small bugs and corrections as I see them, for quickfixing

Please add to list any other little frontendy things as you see them, will check off as they are patched.

lfeine commented 5 years ago

(replying here from #83 ) I'm really not familiar with bibtex, unfortunately. If it can wait about a week, I can look into it if you want.

interrogator commented 5 years ago

No it's OK @lfeine , I'm sure I will have time at some point to do it! Just thought maybe you'd already be a bibtex person :)

martindusinberre commented 5 years ago
martindusinberre commented 5 years ago
martindusinberre commented 5 years ago
interrogator commented 5 years ago

Thanks, these are all easy to do.

Quick note however @martindusinberre, you asked me to go use some initiative and flesh out paragraphs with new text where I can. But so far, even the smallest of these changes has met with (totally understandable) resistance, generally based on a discussion that was had before I arrived on the project. I can't divine the outcomes of these discussions, and am pretty sure that most things I add in the future will meet the same fate (it has happened about 4 times since Friday). So for now, I'm happy to give back the 'artistic licence', and restrict the stuff I do without checking with you prior to spelling, grammar, formatting. I can add text that you write (e.g. for About/FAQ etc), and also to collect my opinions on changes I would like, such as my issue the name 'topic' vs 'game' and so on ... but coming up with content and then reverting it is not a good use of time :nerd_face:

interrogator commented 5 years ago

Everything ticked off in this thread is in frontend master branch btw

martindusinberre commented 5 years ago

@interrogator great. About the editing process more generally: Re "your games": mine is a question, not a shut-down. If you think the second paragraph you added should stay even in light of the question I asked, then please just answer why you think so, and leave it in. That's why I also asked @Helena-LiT for her opinion. Re. "topic selection": this is something you and I discussed on rocketchat last Friday, where I explicitly said that we wanted to keep mention of "marugoto" away from the player except in the "about" tab. That was a decision. If you think it's wrong, please reopen a discussion but don't just make the change as if we never had a discussion. More generally about your perception that I'm shutting your ideas down: this thread is not the place for you to air that perception. We have a meeting tomorrow at 10am--let's talk about it then.

interrogator commented 5 years ago

I don't think it's 'shutting down', don't worry --- it's just that it turns out that you guys have thought through a lot of the stuff I find myself thinking through, it ends up duplicating efforts, reverting changes, and so on. But yeah we can find a solution to the thing tomorrow if you still think it is necessary to have me trying to create content for the main page. I just think focussing on edits, in addition to coding, might be a better use of time at present. To discuss.

martindusinberre commented 5 years ago

Great, good plan. Definitely sounds like a good idea to avoid duplicatio. See you tomorrow, M

martindusinberre commented 5 years ago

@lfeine or @interrogator

interrogator commented 5 years ago

Can do if need be, but might be a good practice run for @lfeine ?

lfeine commented 5 years ago

Good idea, I'll do that this weekend then.

martindusinberre commented 5 years ago

@interrogator

For the time stamp in the Notebook, can we set it to Central European Time? At the moment it's on Greenwich Mean Time. Not a major problem if it's a lot of work, though.

martindusinberre commented 5 years ago
interrogator commented 5 years ago

same-size

interrogator commented 5 years ago

Just a note on the above --- there are classes like lead/lead-text which one would think should be used for the first paragraph of text on pages like this. But such classes make the paragraph bigger, and it looks nasty when there is a second smaller paragraph underneath. Best solution is to simply never use the lead type classes when styling these content blocks. The above is how it looks with no lead classes and with a <br/> between paragraphs. This info may help @lfeine in formatting various text blocks.

Note, we're trying not to mess with the existing CSS, because doing that will cause weird stuff to happen inside the game. For example, <a href=... elements won't get styled with underlines because of the existing CSS. If you want underlines, do <a class="underlined" href=.... i.e., If you need anything extra in terms of CSS, best way is always to create new classes, rather than modifying what is already there.

interrogator commented 5 years ago
* [ ]  Can @interrogator suggest in which tab on the front page we add the following sentence (or something similar)? "Every effort has been made to check copyright for the images appearing in Lives in Transit, and to provide bibliographic links. Please contact the LiT Team if you spot any inadvertent errors in acknowledgement."

One solution, we could have it in FAQ, for a question like How is Lives in Transit licensed?? Some other stuff could go in there as well... @martindusinberre how does that sit with you? Helps flesh out FAQ section, which we need right now

interrogator commented 5 years ago
martindusinberre commented 5 years ago

@interrogator great, please put that sentence in FAQ then, along with the licencing stuff. And who was giving me grief for US english the other day ? :) "licensing"...

interrogator commented 5 years ago

@martindusinberre on the select topic screen, can we change:

It isn't a preview (anymore)...

martindusinberre commented 5 years ago

@interrogator Yes, good idea. Thanks.

interrogator commented 5 years ago
martindusinberre commented 5 years ago

@interrogator

martindusinberre commented 5 years ago

@interrogator Sorry, I now realise we've managed to overlook explaining "Marugoto" on the "About" tab. My suggestion (please edit/rewrite/restructure as you see fit):

martindusinberre commented 5 years ago
martindusinberre commented 5 years ago
interrogator commented 4 years ago

@interrogator

* [ ]  Change the time stamp settings in "Your games". This is because when I exited a game halfway through just now, "Open Games" recorded my unfinished game as being played on 7 December, not 7 November!

* [ ]  For the time stamp in the Notebook, can we set it to Central European Time? At the moment it's on Greenwich Mean Time. Not a major problem if it's a lot of work, though.

This gets its own issue, so closing here

interrogator commented 4 years ago

Closing this long thread. Will make a new one. Great job @martindusinberre @lfeine