OpenDataServices / org-ids

Front end application for http://org-id.guide
http://org-id.guide
Other
17 stars 9 forks source link

Homepage `sector` select doesn’t update browser bar URL #160

Open andylolz opened 7 years ago

andylolz commented 7 years ago

On the homepage, window.location is updated when selects are set. Funnily enough, it looks like it happens as a side effect of this bit of code: https://github.com/OpenDataServices/org-ids/blob/4603a4991e65e18f0b42a8439ef61bd65d8c62d9/prefix_finder/frontend/static/js/main.js#L25-L38

Since ‘sector’ doesn’t have sub-options, the sector select doesn’t update the URL, so it’s inconsistent with the other selects.

I’d question the need to set window.location at all, but in any case it should be consistent.