In T232653#5492187, @Jdlrobson wrote:

@ovasileva @alexhollender how do you feel about removing the uploads page altogether?

In T233166#5503382, @Jdlrobson wrote:

@alexhollender question for you in developer notes

In T233171#5503006, @Esanders wrote:

The word loading is not centered in the header

This is as designed.

In T231613#5501336, @Jdlrobson wrote:

@alexhollender should the star be blue or green? (it's easier if it's blue..)

In T229440#5499327, @Edtadros wrote:

@alexhollender please take a look at the items with ❓ .

In T201687#5497297, @Jdlrobson wrote:

What is the problem we're left to solve now? From the look of it everything looks fixed to me. @alexhollender was there anything specific you wanted to do here?

In T232652#5500305, @pmiazga wrote:

@Alex I just want to confirm - those mocks are not in sync, right? We do not show a different toolbar for Android and iOS.
The Android mock shows the mobile mode in the AMC mode without overflow menu (icons are spread through the width of toolbar - same margins for all icons)
The iOS mock shows the mobile mode in regular mode - language icon is on left, all icons goes to the right.

Spoke with @pmiazga, the issue with the Languages label is related to icon work.

@Jdlrobson @Jdrewniak unless there's a caching issue it looks like Watch and Un-watch (which appear in the article toolbar) have not been updated yet. When inspecting them I see the old SVG.

@Jdlrobson @Jdrewniak this is fixed. Currently the icons are slightly closer together than in the spec, however if we increase the icon areas to 44px (task coming soon) then it will be all set. I don't think we should do any remaining work here.

Fixed. Screenshot from iPhone 5.

This has been fixed

Fixed

@Edtadros this seems to be fixed

@Jdlrobson @Jdrewniak the positioning of the menus is correct, but the vertical spacing between the list items is still too large. I would recommend 0.75em for the vertical padding, e.g.

.toggle-list-item {
    display: block;
    padding: .75em 0.875em;
}

@Edtadros as you pointed out there's a new issue with this element, however since the issue described here has been fixed I suggest we resolve this task. I will create a new one to deal with the alignment issue you noted:

There is still an issue with this element, however the issue described in the task has been fixed so I suggest we close this and I will open a new task to address the gap at the bottom of the header, which may be related to T215426, and the fact that the "Loading" element is not vertically centered.

@RHo I can't quite tell from the screenshot you posted, but super minor consistency thing — the underline is meant to overlap the border beneath it by 1px:

On Slack we discussed the need for a flush-top class to fix this, which also seems to be the issue with the reference drawer alignment

@pmiazga the icon and label are fixed. I noticed that the languages label is now dark gray instead of light gray, is this related to the change here?

Colors have been fixed. This doesn't require addition QA.

This is still an issue

@jbond thank you, I am now able to access Turnilo.

In T18691#5488601, @Msftwikipmey wrote:

#2 - Does "deployment on WMF systems" mean to make this part of MediaWiki core software? I think this has always been the expectation, which is why it has gone through many iterations of design, i18n and perf improvement :) Could WMF friends chime in to confirm this is still the case?

In T232284#5484770, @Jdlrobson wrote:

Yeh that could work. How would it look with undo and revert? Would buttons be stacked vertically or horizontally?

@Jdlrobson also seeing it like this sometimes

In T232652#5488286, @Jdlrobson wrote:

Where does history link to? The AMC version or the current mobile simplified version?

In T230034#5490742, @Jdrewniak wrote:

after icon changes you get a square, hurraz.

@Jdlrobson @Jdrewniak here are the issues I'm seeing on beta:

@Jdlrobson apologies for not clarifying this initially: on History and Contributions the user icon should be #72777d. On the diffs page it's good as is.

Looks good on iOS/Safari

I wonder if the treatment of Undo on the History page would work here? I'm also fine with parentheses.

@Jony my LDAP account is now linked to my Phabricator account

@ovasileva @Jdlrobson are we tracking this elsewhere already?

@Urbanecm thanks for raising this. I understand the issue. Marking this as low since we have no current plans to work on mobile talk pages unfortunately. Maybe in scope for the editing team's upcoming project, cc @ppelberg

In T232011#5479836, @ovasileva wrote:

@alexhollender - looks like we won't be able to fix this until Sept 19th (next week's train) - is that okay with you?

@ovasileva not sure where this should live for now. I feel like the scope is larger than just a design question. Thoughts?

@Esanders to clarify the problem here — for people on slow connections:

Ok, shell account name is ahollender.

In T232383#5477186, @Volker_E wrote:

No strong preference one, one thing though is, the mobile site doesn't use an indicated list, only indentation. But I think that is acceptable.

@Volker_E not sure what we're doing with this task. Are there visual implications here? If not can we remove it from the design board?

We didn't end up using this task. The work was done in various other tasks.

In T232476#5479232, @jbond wrote:

@alexhollender what is your shell username i don't see anything obvious

@elukey I was told that creating this task was the proper way to gain access to LDAP > Turnilo. Please let me know if there is a preferred way to request that.

@Volker_E perhaps this?

I don't think this needs to be done in conjunction with the current icon work.

@Jdlrobson the filled user icon refers to these two:

In T230692#5425843, @Niedzielski wrote:

Is this device specific or am I misunderstanding?

@Jdrewniak I checked out the codepen, looks great.