Skip to content
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

let maps show some interesting data #4

Open
thoka opened this issue May 3, 2015 · 11 comments
Open

let maps show some interesting data #4

thoka opened this issue May 3, 2015 · 11 comments

Comments

@thoka
Copy link
Member

thoka commented May 3, 2015

currently maps point to more or less nowhere

let them point to places, where some data exists

@species
Copy link

species commented May 4, 2015

I've commited better places in d4860b7, please deploy (as I am not able to run docpad atm).

@thoka
Copy link
Member Author

thoka commented May 4, 2015

deployed

On Mon, May 4, 2015 at 3:06 PM, Michael Maier [email protected]
wrote:

I've commited better places in d4860b7
d4860b7,
please deploy (as I am not able to run docpad atm).


Reply to this email directly or view it on GitHub
#4 (comment)
.

@species
Copy link

species commented May 11, 2015

which branch/fork is currently deployed on transformap.co?
the text below the maps links correctly, but the images itself still to zoomlevel 3 - but i cannot find matching source files anywhere :-/

@almereyda
Copy link
Member

Have a look into
https://github.com/TransforMap/transformap.github.io/blob/source/src/render/index.html.eco

On 11 May 2015 at 09:49, Michael Maier [email protected] wrote:

which branch/fork is currently deployed on transformap.co?
the text below the maps links correctly, but the images itself still to
zoomlevel 3 - but i cannot find matching source files anywhere :-/


Reply to this email directly or view it on GitHub
#4 (comment)
.

@species
Copy link

species commented May 12, 2015

WTF?!? Apart from broken HTML (unclosed -tags, 's inside 's ) in there, framework and content was mixed up badly.
Fixed in 2db58fa.
Please test and deploy :-)

@almereyda
Copy link
Member

Please calm down.
You know 36b9d07 had been added in the last seconds before departing to the train.

The mix of templating and content is now even extended, that the *.md files carry plain HTML.

As long as we don't automatically create the maps list, fighting for strict seperation here could mean overoptimization.

Just put the links in the template maybe?
So logic (links) will be seperate from content (image, text).
In the end we still align the partials manually to their respective maps.


@thoka @species

As of keikreutler#5 (comment) I want to revert 8a83052 so we can switch to a Pull Request driven merge-and-deploy workflow as we agreed.

@almereyda
Copy link
Member

@gandhiano @species @thoka @keikreutler
For comparison:
This is how bad mixing of the MVCU pattern looks like.
~ advise ~ scrub the presentation at least (hover > onmousemove)

4th_try_in_der_gruppe_ecobytes_co-munity_-_2015-05-15_02 24 14
Note: co-munity.net is based on Open Atrium 2, which is based on drupal, which is written in PHP.

@gandhiano
Copy link
Member

What you see has nothing to do with PHP, Drupal or Open Atrium, but rather missing configuration and testing of the input filters (only the standard WYSIWYG has been somewhat thoroughly tested).

Also don't understand what this has to do with the concrete issue.

@almereyda
Copy link
Member

But the MarkDown input filter is not implemented in PHP and runs on Drupal used by a module in your Open Atrium instance? Or did you mean it's implemented in JavaScript and runs only client side, after rendering the DOM?
Then how come the drupal HTML leaks into it?

For relation to the issue: I wanted to give an example where a bad separation of models and views, which is by definition already artificial, leads to unwanted side effects.

@gandhiano
Copy link
Member

Markdown in Drupal is a text filter, which can be used in an input format. The filter just does some string conversion and stores it in the database (and reads from the database to prepare the proper HTML). The input format (which in the case I implemented is also called MarkDown) is a combination of different filters. Input formats can also have their own frontends/editors - this is where I want to extend the current crappy editor by setting up this apparently nice Markdown Editor for BUEeditor with AJAX support for preview without reload (I guess with some coding one could even get the Markdown to automatically display on another form on the same page)

The reason why you have the HTML code appearing there has probably to do with a (automatic) switching between input formats, due to a faulty implementation, at least in the discussion replies (not sure if OA related, or my own setup of input formats, but will try to address this on the next update).

In this sense, I think Drupal makes here a quite good separation of models and views - the models are provided here by the filters, the views by the editors. Input filters are mostly model, but make the bridge to specific views.

@species
Copy link

species commented May 22, 2015

Thanks for deploying! ☺
May I close this issue, or do you need it for discussions?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants