1 2013-07-23T06:01:13 *** rciorba
2 2013-07-23T06:43:27 *** rciorba
3 2013-07-23T06:47:19 *** rciorba
4 2013-07-23T06:55:19 *** rciorba
5 2013-07-23T06:55:25 *** rciorba
6 2013-07-23T08:25:24 *** rciorba
7 2013-07-23T08:53:39 <ThomasWaldmann> moin
8 2013-07-23T08:59:05 <TheSheep> sharky93: yes, please commit to the -bootstrap repo
9 2013-07-23T08:59:39 *** greg_f
10 2013-07-23T09:16:03 <ananasova> moin
11 2013-07-23T09:23:11 *** WarP|work
12 2013-07-23T10:39:29 *** WarP|work
13 2013-07-23T10:55:30 *** WarP|work
14 2013-07-23T12:30:32 <sharky93> moin
15 2013-07-23T12:34:06 <sharky93> TheSheep: made the commit, is it ok?
16 2013-07-23T12:37:55 <spy__> moin
17 2013-07-23T12:38:19 <spy__> ThomasWaldmann: dreimark: quick cr https://codereview.appspot.com/11715043/
18 2013-07-23T12:39:58 *** dave_largo
19 2013-07-23T13:28:03 <TheSheep> sharky93: doesn't look too good :/
20 2013-07-23T13:28:54 <TheSheep> sharky93: can you work a little on the layout of elements?
21 2013-07-23T13:43:32 <sharky93> TheSheep: the general meta portion looks odd to me, same with you?
22 2013-07-23T13:45:49 <TheSheep> sharky93: it's all just a bunch of form fields stringed together on a page
23 2013-07-23T13:45:58 <TheSheep> sharky93: it should have more structure
24 2013-07-23T13:53:52 <sharky93> TheSheep: right. also how about using "info labels" from here @ http://twitter.github.io/bootstrap/components.html#labels-badges rather than just labels for the form fields?
25 2013-07-23T14:03:57 <TheSheep> sharky93: what for?
26 2013-07-23T14:04:09 <TheSheep> sharky93: do you need the color to indicate something?
27 2013-07-23T14:05:33 <sharky93> TheSheep: just doing the job of a label in a more visible fashion
28 2013-07-23T14:05:51 <TheSheep> sharky93: communicate, not decorate
29 2013-07-23T14:06:28 <sharky93> TheSheep: right :)
30 2013-07-23T14:06:46 <TheSheep> sharky93: please think about the layout of that view, where to put each element and how they should be grouped
31 2013-07-23T14:07:07 <sharky93> TheSheep: sure, ok
32 2013-07-23T14:50:58 <sharky93> TheSheep: some update, using tabs, please have a look
33 2013-07-23T15:13:55 <ThomasWaldmann> spy__: /me looking...
34 2013-07-23T15:19:10 <ThomasWaldmann> spy__: see comment there
35 2013-07-23T15:25:34 <ThomasWaldmann> sharky93: what is the "general meta portion"?
36 2013-07-23T15:26:51 <sharky93> ThomasWaldmann: lists the basic meta?
37 2013-07-23T15:26:58 <sharky93> right should be named, basic meta :)
38 2013-07-23T15:27:34 <ThomasWaldmann> i need a url and a position on the screen
39 2013-07-23T15:29:49 <sharky93> coming right up
40 2013-07-23T15:30:40 <sharky93> ThomasWaldmann: this should help @ http://imagebin.org/265365
41 2013-07-23T15:30:48 * ThomasWaldmann has the repo clone running here
42 2013-07-23T15:31:15 <sharky93> ThomasWaldmann: have you pulled the latest changes?
43 2013-07-23T15:31:45 <ThomasWaldmann> yes
44 2013-07-23T15:32:01 <ThomasWaldmann> you still did not say the url
45 2013-07-23T15:32:44 <sharky93> http://localhost:8080/+modify/Home
46 2013-07-23T15:32:54 * sharky93 is a bit confused. sry
47 2013-07-23T15:33:28 <ThomasWaldmann> ok, now i see it.
48 2013-07-23T15:34:31 <sharky93> is it any good to use? :)
49 2013-07-23T15:34:50 <sharky93> *better
50 2013-07-23T15:37:48 <ThomasWaldmann> somehow that view switching tabs Edit/general meta/extra meta are strangely located.
51 2013-07-23T15:38:00 *** WarP|work|alt
52 2013-07-23T15:38:48 <ThomasWaldmann> somehow too much nav everywhere for a page you are not supposed to leave anyway except by save/cancel
53 2013-07-23T15:39:54 <ThomasWaldmann> that fake "Help" displayed at the bottom could go into the same switch, so it would be Edit/general meta/extra meta/help
54 2013-07-23T15:40:17 *** WarP|work
55 2013-07-23T15:42:28 <sharky93> feels like trolling the user, opena switch labelled as help and bam :P
56 2013-07-23T15:46:51 <ThomasWaldmann> "bam"?
57 2013-07-23T15:47:51 <sharky93> "bam" as in .. the sudden realisation after that message pops up on the screen -> "There is no help, you're doomed!"
58 2013-07-23T15:48:58 <ThomasWaldmann> well, of course we need a real text there. but that has nothing to do with the placement of it and that a tab is maybe better than just putting it below the editor window (which could be quite a lot to scroll)
59 2013-07-23T15:49:54 <sharky93> yeah, i agree :) .. was just pointing out about the odd message :)
60 2013-07-23T15:50:39 <ThomasWaldmann> iirc we have an issue for that already, but fixing that is not part of the UI project
61 2013-07-23T15:51:20 <ThomasWaldmann> btw, concerning issues on your issue tracker: can you please always identify issues that are easy to fix and fix them first?
62 2013-07-23T15:51:38 <ThomasWaldmann> just to keep the amount of issues low
63 2013-07-23T15:55:44 <sharky93> will do.
64 2013-07-23T15:57:11 <ThomasWaldmann> sharky93: is it clear what the difference between path and location breadcrumbs is? and how the use cases are?
65 2013-07-23T15:58:17 <sharky93> ThomasWaldmann: i think i havent seen the location breadcrumb in action yet, do we have it in the main repo?
66 2013-07-23T15:58:51 <ThomasWaldmann> it should work in the other themes, yes
67 2013-07-23T15:59:45 * sharky93 is amazed how did he manage to avoid it
68 2013-07-23T16:00:29 <xiaq> ThomasWaldmann: PR: https://bitbucket.org/thomaswaldmann/moin-2.0/pull-request/176 :)
69 2013-07-23T16:00:37 <ThomasWaldmann> the location breadcrumbs are just displaying where you are (== the item name) and if that is e.g. MainPage/SubPage/SubSubPage, each of the 3 components link to the respective page in the hierarchy
70 2013-07-23T16:01:21 <sharky93> ThomasWaldmann: so SubPage links to MainPage/SubPage .. ?
71 2013-07-23T16:01:40 <ThomasWaldmann> sharky93: yes.
72 2013-07-23T16:02:19 <ThomasWaldmann> sharky93: and also realized that "/" is used as separator for the location components (as usual)
73 2013-07-23T16:02:20 <sharky93> ThomasWaldmann: TheSheep i think we should use only one among them, either location bcs/path bcs .. what do you think?
74 2013-07-23T16:02:22 <ThomasWaldmann> -d
75 2013-07-23T16:02:33 <sharky93> :D
76 2013-07-23T16:02:57 <ThomasWaldmann> sharky93: you used "/" in the trail (path breadcrumbs), which was very confusing for me as I first thought that shall be the location breadcrumbs
77 2013-07-23T16:04:19 * xiaq accidently introduces some changes that don't belong...
78 2013-07-23T16:04:27 <ThomasWaldmann> trail should use something like ">>" to indicate that this is about path, not location ("/")
79 2013-07-23T16:04:41 <xiaq> https://bitbucket.org/xiaq/moin-2.0/commits/b36d86da94fc75ad64f4d4b8573e1cad072a39c8 at the bottom, MoinMoin/templates/ticket/base.html - shouldn't be in this changeset
80 2013-07-23T16:04:50 <xiaq> ThomasWaldmann: what's the best thing to do now... ?
81 2013-07-23T16:05:46 <ThomasWaldmann> xiaq: well usually there is no way back after publication. but the likelihood that anyone except me pulls that is rather low, so....
82 2013-07-23T16:06:42 <ThomasWaldmann> (you can either strip it and do it again [under such circumstances] or fix it by another commit [less pretty, but works always])
83 2013-07-23T16:11:43 * xiaq rewrote history, striped changesets and resent PR https://bitbucket.org/thomaswaldmann/moin-2.0/pull-request/177/3-changesets/diff
84 2013-07-23T16:14:19 <ThomasWaldmann> btw, you can always have a look at hg status / hg diff before commit and hg out before push
85 2013-07-23T16:18:14 <ThomasWaldmann> guess we'll need such a "display widget" for contenttype soon. usual CTs are not very user-friendly and also too long, breaking the layout of sidebars.
86 2013-07-23T16:19:30 <ThomasWaldmann> hmm, we even have a simiar thing, but in other direction: when we present the contenttype chooser when creating a new item
87 2013-07-23T16:36:21 *** WarP|work
88 2013-07-23T16:39:52 *** WarP|work|alt
89 2013-07-23T16:44:13 <ThomasWaldmann> hmm, TheSheep sharky93: there could be one way to combine location and path breadcrumbs. but i somehow fear this is too complex:
90 2013-07-23T16:44:46 <ThomasWaldmann> old3 >> old2 >> old1 >> currentmain / currentsub / currentsubsub
91 2013-07-23T16:45:46 <ThomasWaldmann> esp. if one considers the we also have interwikiname and namespace...
92 2013-07-23T16:46:43 <ThomasWaldmann> a dropdown for the old stuff like browsers offer it for url history could also work
93 2013-07-23T16:59:12 *** WarP|work
94 2013-07-23T17:03:53 <ThomasWaldmann> sharky93: about "site navigation" section at the left vs. "site actions" at the right:
95 2013-07-23T17:05:47 <ThomasWaldmann> for the right one, i think we want to express "item related navigation"
96 2013-07-23T17:06:28 <ThomasWaldmann> "site" sounds rather global, but that index is a subitems index. that sitemap is the link structure as seen when starting from the current item.
97 2013-07-23T17:07:13 <ThomasWaldmann> also, it is of course rather "navigation" than an "action"
98 2013-07-23T17:08:46 <ThomasWaldmann> "item related navigation" might be a bit too wide for a sidebar, so it should be a bit shorter
99 2013-07-23T17:21:40 *** greg_f
100 2013-07-23T19:22:49 <sharky93> ThomasWaldmann: i wonder how the idea of "burn"ing a disk developed, since that preserves information :P ;)
101 2013-07-23T19:25:01 <sharky93> thanks for the help with the issues :)
102 2013-07-23T19:29:36 <ThomasWaldmann> sharky93: that depends on the temperature, deformation and contamination of the disk. i would bet on something there. :)
103 2013-07-23T19:31:16 *** brunomartin
104 2013-07-23T21:04:56 *** dave_largo
105 2013-07-23T21:05:58 <sharky93> TheSheep: too bad bootstrap doesnt have support for "input type=file" :(
106 2013-07-23T21:20:08 <ThomasWaldmann> sharky93: maybe integrate the jquery-file-upload we use at another place (instead of the button) ?
107 2013-07-23T22:01:36 <ThomasWaldmann> sharky93: still there?
108 2013-07-23T22:04:17 *** rciorba
109 2013-07-23T22:43:31 <sharky93> ThomasWaldmann: yeah, just got back.
110 2013-07-23T22:46:38 <sharky93> ThomasWaldmann: yeah, that is an option, but imho think we should find a way to make the input type="file" less ugly, since that is a more standard way.
111 2013-07-23T22:47:34 * ThomasWaldmann updated repo and filed some issues
112 2013-07-23T22:51:35 <sharky93> TheSheep: please have a look at the modify view (updated repo) and some comments too at its issue tracker :)
113 2013-07-23T22:55:48 * sharky93 says gn
114 2013-07-23T22:56:06 <ThomasWaldmann> sharky93: what was the reason to move the upload button?
115 2013-07-23T22:57:09 <sharky93> ThomasWaldmann: aligning the textarea with the tabs
116 2013-07-23T22:59:03 <ThomasWaldmann> ok, we likely will have to reconsider that, but maybe after solving the dual scrollbar issue
117 2013-07-23T22:59:24 <ThomasWaldmann> why is the comment input so narrow?
118 2013-07-23T23:00:20 <sharky93> ThomasWaldmann: im not able to get a predefined size to work for it ..
119 2013-07-23T23:01:55 <ThomasWaldmann> oh?
120 2013-07-23T23:02:08 <ThomasWaldmann> file an issue to get feedback?
121 2013-07-23T23:02:11 <sharky93> like, http://imagebin.org/265427 and http://imagebin.org/265428 are the oterh options
122 2013-07-23T23:02:14 <sharky93> *other
123 2013-07-23T23:03:04 <ThomasWaldmann> well, it seems to work
124 2013-07-23T23:03:47 <ThomasWaldmann> maybe putting the OK button to the left of the comment input would make it look less "lost" there at the right
125 2013-07-23T23:04:01 <sharky93> lets see
126 2013-07-23T23:05:44 <ThomasWaldmann> ok, lets see later today :)
127 2013-07-23T23:07:11 <sharky93> :P
128 2013-07-23T23:07:25 <sharky93> ThomasWaldmann: well, does this look okay? http://imagebin.org/265429
129 2013-07-23T23:07:29 <sharky93> i think this works
130 2013-07-23T23:12:06 <sharky93> anyways gn :)
131 2013-07-23T23:13:23 <ThomasWaldmann> almost. of course i meant to the right of the button, not below it.
132 2013-07-23T23:38:02 <ThomasWaldmann> TheSheep: do you know jasny bootstrap extensions?
