{"issue":{"created_at":"2011-08-01T12:16:05Z","state":"open","updated_at":"2011-11-20T12:36:05Z","user":{"gravatar_id":"5797e5e2ea246eb42e1c586575daab9d","id":922874,"url":"https://api.github.com/users/SammysHP","login":"SammysHP","avatar_url":"https://secure.gravatar.com/avatar/5797e5e2ea246eb42e1c586575daab9d?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"comments":55,"body":"It's one of the main parts of c:geo: The detail view. Over the time a lot of things were added and now this view is damn unclear.\r\n\r\n\r\n\r\nHere a few screenshots from as-is:\r\n\r\n1) http://www7.pic-upload.de/01.08.11/sqbuymk78at.jpg\r\n2) http://www7.pic-upload.de/01.08.11/bcrm23zsqpch.jpg\r\n3) http://www7.pic-upload.de/01.08.11/eeajou8dh9a.jpg\r\n4) http://www7.pic-upload.de/01.08.11/l2bgjuromxbx.jpg\r\n\r\n1) First page, a lot of details that nobody is interested in.\r\n2) Second page, oh, there's a description at the end.\r\n3) Third page, finally, the long desired hint.\r\n4) The complete disaster. Important parts highlighted.\r\n\r\n\r\n\r\nSo I've made a draft of how it could look like. Sure, some things could be improved but it's a first try.\r\n\r\nhttp://www7.pic-upload.de/01.08.11/w1jb3zodruc.jpg\r\n\r\nThe main-changes:\r\n- Only the most important details are shown at the top. The old style can be displayed by clicking on the details. Not the small direction-arrow that is missing at the moment.\r\n- All sections are collapsible and are collapsed at beginning. You can show/hide them by clicking on the title (maybe this could be improved, because a lot of titles looks ugly)\r\n- Empty sections are hidden.\r\n- We use icons for attributes. I don't know who is the right-holder, maybe we must make our own icons. A click at them will bring back the text view.\r\n\r\n\r\nI'm not so familar with layouts in Android so it would be nice if someone can make it or help me with it.","assignee":{"gravatar_id":"5797e5e2ea246eb42e1c586575daab9d","id":922874,"url":"https://api.github.com/users/SammysHP","login":"SammysHP","avatar_url":"https://secure.gravatar.com/avatar/5797e5e2ea246eb42e1c586575daab9d?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"id":1322454,"pull_request":{"patch_url":null,"html_url":null,"diff_url":null},"number":83,"labels":[{"name":"frontend design","color":"d7e102","url":"https://api.github.com/repos/cgeo/c-geo-opensource/labels/frontend+design"}],"title":"Redesign detail-view","closed_at":null,"html_url":"https://github.com/cgeo/c-geo-opensource/issues/83","milestone":null,"url":"https://api.github.com/repos/cgeo/c-geo-opensource/issues/83"},"action":"created","comment":{"created_at":"2011-11-20T12:36:05Z","updated_at":"2011-11-20T12:36:05Z","user":{"gravatar_id":"5797e5e2ea246eb42e1c586575daab9d","id":922874,"url":"https://api.github.com/users/SammysHP","login":"SammysHP","avatar_url":"https://secure.gravatar.com/avatar/5797e5e2ea246eb42e1c586575daab9d?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"body":"Why? Because at the moment we have a lot of flags, global variables etc. It's really difficult to split and refactor the code.\r\n\r\nSure, I would never push such a large change to master. It's in my fork in the branch ```viewpager```.\r\n\r\nCurrently 500 lines are done. A lot of unnecessary and weird code was in the old implementation. The functionality should be the same, but now much more cleaned up. Should be better understandable if it's finished.\r\n\r\nAlso I'm making my changs in a new class, ```CacheDetailActivity```.","id":2803685,"url":"https://api.github.com/repos/cgeo/c-geo-opensource/issues/comments/2803685"}}