<blockquote>
<p>I've always wanted to move browser to the client side so that we don't read all the data twice</p>
</blockquote>
<p>That's a non-sequitur though. We could avoid reading all the data twice by just responding once with all the html (and some json for the map view highlighting) in the first place.</p>
<p>Making the browse pages entirely reliant on the API means that for a given request, the browser will be requesting info from the server in <em>deliberately the wrong format</em> and then having to convert it, instead of the server just responding with the same information in the format required in the first place.</p>
<p>I feel that this whole "make everything client side" and "sling around API responses and parse them on the client using javascript" has few tangible advantages and is mainly being done "because architecture". We can already see the downsides of this with the translations - even split up per-language, we still need to get each client to download a bunch of stuff that will likely never be shown to them.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/issues/1949#issuecomment-413775505">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABWnLerktXSLQeGDwtxvucuhqTCLqg44ks5uRmkAgaJpZM4V_j_f">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABWnLY_EdcQkjxoz44veYuA9RwAezlFiks5uRmkAgaJpZM4V_j_f.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/openstreetmap/openstreetmap-website","title":"openstreetmap/openstreetmap-website","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/openstreetmap/openstreetmap-website"}},"updates":{"snippets":[{"icon":"PERSON","message":"@gravitystorm in #1949: \u003e I've always wanted to move browser to the client side so that we don't read all the data twice\r\n\r\nThat's a non-sequitur though. We could avoid reading all the data twice by just responding once with all the html (and some json for the map view highlighting) in the first place.\r\n\r\nMaking the browse pages entirely reliant on the API means that for a given request, the browser will be requesting info from the server in *deliberately the wrong format* and then having to convert it, instead of the server just responding with the same information in the format required in the first place.\r\n\r\nI feel that this whole \"make everything client side\" and \"sling around API responses and parse them on the client using javascript\" has few tangible advantages and is mainly being done \"because architecture\". We can already see the downsides of this with the translations - even split up per-language, we still need to get each client to download a bunch of stuff that will likely never be shown to them."}],"action":{"name":"View Issue","url":"https://github.com/openstreetmap/openstreetmap-website/issues/1949#issuecomment-413775505"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/issues/1949#issuecomment-413775505",
"url": "https://github.com/openstreetmap/openstreetmap-website/issues/1949#issuecomment-413775505",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [openstreetmap/openstreetmap-website] Reduce size of application.js by removing translations? (#1949)",
"sections": [
{
"text": "",
"activityTitle": "**Andy Allan**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@gravitystorm",
"facts": [

]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueComment\",\n\"repositoryFullName\": \"openstreetmap/openstreetmap-website\",\n\"issueId\": 1949,\n\"IssueComment\": \"{{IssueComment.value}}\"\n}"
}
]
},
{
"name": "Close issue",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueClose\",\n\"repositoryFullName\": \"openstreetmap/openstreetmap-website\",\n\"issueId\": 1949\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/openstreetmap/openstreetmap-website/issues/1949#issuecomment-413775505"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 368984031\n}"
}
],
"themeColor": "26292E"
}
]</script>