This project is mirrored from https://github.com/weirdgloop/mediawiki-extensions-MobileFrontend. Pull mirroring updated .
  1. 19 Mar, 2019 11 commits
  2. 18 Mar, 2019 7 commits
    • jdlrobson's avatar
      Prepare for a talk topic form component · 44c8794a
      jdlrobson authored
      In future this talk overlay will be a standard overlay with a talk
      topic form component inside it. While the template will be removed
      shortly as part of this factory this sets the direction and makes sure
      the CSS is ready.
      
      Bug: T217102
      Change-Id: I58e5b3ad61871d456e6b26de3d9027809b11df2e
      44c8794a
    • jenkins-bot's avatar
      Merge "Don't duplicate panel rules" · 711628b0
      jenkins-bot authored
      711628b0
    • jenkins-bot's avatar
    • Translation updater bot's avatar
      Localisation updates from https://translatewiki.net. · bf74ebd8
      Translation updater bot authored
      Change-Id: I03a11bd9668a911089a1ae134de726dc53ce40b8
      bf74ebd8
    • jdlrobson's avatar
      Generalise a CSS rule for paragraphs within panels · fa8523ea
      jdlrobson authored
      Bug: T217102
      Change-Id: Id2993dbaa69ea8df0db922453f1bbfed91c3d9ee
      fa8523ea
    • jdlrobson's avatar
      Drop unused View parameter · e77fb5e8
      jdlrobson authored
      No longer a thing since Ie86067c6f81deebfd85efbb9537519ad7179f5a4
      
      Bug: T217102
      Change-Id: I13ca0d468cb92ef0c6a61a54a0a6b33d0fe8d55c
      e77fb5e8
    • Nicholas Ray's avatar
      Add new presentational component: imageDetails.js · 6f56b3c5
      Nicholas Ray authored
      This is a `dumb` presentational component that merely presents details
      about an image including the author, license link, details link, and
      caption. It only has rendering logic.
      
      This will be used by image carousel in
      I549add34101d58439b25265473a3a611c3324154. In the meantime, the current
      ImageOverlay will use its styles found in imageDetails.less.
      
      Additional changes:
      
      * The fixme instructing to merge with overlay-footer-container was removed
      
      * ImageOverlay.hogan was modifed to use image-details styles
      
      * ImageOverlay.js was modified to use image-details styles
      
      * .animations .media-viewer .details was removed from mediaViewer.less.
      This transition is assumed safe to remove because opacity isn't even a
      property of .details (now .image-details)!
      
      * extension.json was modified to use imageDetails.less
      
      * ImageOverlay.test.js was fixed to support .image-details
      
      Bug: T216198
      Change-Id: Ia227186ccc1d8200e8357ea411358347f5b4339b
      6f56b3c5
  3. 17 Mar, 2019 1 commit
  4. 15 Mar, 2019 5 commits
  5. 14 Mar, 2019 11 commits
    • jdlrobson's avatar
      Do not jump to top of text editor during typing · 7b9dbbfa
      jdlrobson authored
      I'm not sure what the history of the EditorOverlay scroll
      onWindowScroll function is, but it seems to be in direct
      conflict with the scrolling hacks disabled
      in I846c0b3013074883ee39d225ed083de03fc6b0c7
      
      After removing this code the editor appears to become usable
      again. This is a great example of why the inheritance pattern is bad.
      It adds lots of indirection and confusion.
      
      Bug: T218062
      Bug: T218352
      Bug: T211490
      Bug: T218062
      Bug: T211491
      Bug: T172877
      Change-Id: I0781a2d1d8252c8c8004f0e8ab67705978ae3c18
      7b9dbbfa
    • Jdlrobson's avatar
      f36fe50a
    • Jdlrobson's avatar
    • Translation updater bot's avatar
      Localisation updates from https://translatewiki.net. · 41ec08dd
      Translation updater bot authored
      Change-Id: Ib786d546782d97fa042b0797cec60dab06455a08
      41ec08dd
    • jdlrobson's avatar
      Disable the iOS scroll emulation in the editor · 3b615e5a
      jdlrobson authored
      We would like to thoroughly test on the beta cluster what the impact of
      disabling iOS emulation is. Due to having issues with local environments
      this is the quickest way to get the widest possible amount of testers
      identifying what the iOS scroll emulation is for.
      
      Change-Id: I846c0b3013074883ee39d225ed083de03fc6b0c7
      3b615e5a
    • jdlrobson's avatar
      Restrict iOS scrolling emulation to overlays that need it · 0d51f475
      jdlrobson authored
      The editor seems to be the only overlay in need of this so I
      think we should disable this behaviour by default. This restores
      the correct scrolling behaviour for the add discussion overlay.
      
      It also separates the concerns more cleanly for scrolling emulation.
      
      Bug: T218069
      Change-Id: If247cee79640e0bbc997feb32fbca4770183a78c
      0d51f475
    • Nicholas Ray's avatar
      Improve carousel API · 708ebdb8
      Nicholas Ray authored
      These changes were made:
      
      * props.featured was thrown out in favor of displaying the item at
      props.items[props.index].
      
      * If props.items[prop.index] is a View, carousel will now recognize that
      and render its $el property. If it is not a View, carousel will pass it
      straight to JQuery's append method. Carousel should not mandate that
      props.items be an array of Views because it might be desirable to have
      the item you want to currently display different from the rest of the
      items in the array. An example of this is by imageCarousel in
      Ibd1cbc6bad2d7d3cf9fffb5e40001e107e7f3333 which will pass Carousel an
      array of async functions but props.items[prop.index] will be a View
      (from promisedImage).
      
      * Tried to improving wording of jsdoc.
      
      Bug: T216198
      Change-Id: I190efb31fa5c0c8308ec880fe2ed04a61ff2af83
      708ebdb8
    • jenkins-bot's avatar
      Merge "Remove slider-button width" · 6a86cc10
      jenkins-bot authored
      6a86cc10
    • jenkins-bot's avatar
      Merge "Stub msg for all Nearby tests" · ef777c50
      jenkins-bot authored
      ef777c50
    • jdlrobson's avatar
      Stub msg for all Nearby tests · bfc3d538
      jdlrobson authored
      Not stubbing this was causing problems in Special:JavaScript/qunit
      _distanceMessage can be called inside _search as well!
      
      Note, we don't load the associated messages in the Special page
      so stubbing is important! jQuery msg was trying to find a $1 in
      the key mobile-frontend-nearby-distance-meters
      
      Bug: T218172
      Change-Id: Ib2b34abff218fc67d1e2b5308928bbf64e1b91fb
      bfc3d538
    • jdlrobson's avatar
      Hygiene: Drop some unnecessary usages of `this.` · d1b0e576
      jdlrobson authored
      These variables are not used outside the scope of this function
      
      Bug: T217296
      Change-Id: I815892260bec1f987b80e386fe5a7a0941c887f6
      d1b0e576
  6. 13 Mar, 2019 5 commits
    • jdlrobson's avatar
      Remove some avoidable inheritance · f8353222
      jdlrobson authored
      Do not extend class properties where parameters can be passed in
      
      This cleans up the NotificationsOverlay a little by making use of
      headerActions, footerActions and heading rather than changing those
      properties via preRender and default extending
      
      Bug: T217296
      Change-Id: I31c86350c31d835fb92a3d74ad4c8aa7e663dc9e
      f8353222
    • jdlrobson's avatar
      Provide a View.make helper method · c8ae79e7
      jdlrobson authored
      Often we'll be wanting to turn lots of jQuery elements into View.
      This helper will make that easier.
      
      It's first usage will be in I815892260bec1f987b80e386fe5a7a0941c887f6
      
      Bug: T217296
      Change-Id: Ifd438dd82550930710b18ce4a4e89b413b25a5a3
      c8ae79e7
    • jenkins-bot's avatar
      8dd40b85
    • jdlrobson's avatar
      Remove slider-button width · 0cfdeb06
      jdlrobson authored
      This rule is overriden by .overlay button and even with that rule
      removed, this has no impact on the visual display due to the min-width
      of 60px which is larger than this width!
      
      Change-Id: I0dd6e4bfaed97a12e1f99ac469edc037eb831675
      0cfdeb06
    • Nicholas Ray's avatar
      Add new presentational component: carousel.js · 6a44a004
      Nicholas Ray authored
      This is a `dumb` presentational component accepting an arbitrary array
      of items that a user should presumably be able to navigate through and a
      featured item that is currently on display. When the user clicks one of
      the navigation arrows, the onSlide callback will give you the next item
      as a param! This component does nothing with the actual changing of
      items - that is left up to other components.
      
      This component will be used in I549add34101d58439b25265473a3a611c3324154
      to display the image carousel. In the meantime, its .slider-button
      styles will be applied to ImageOverlay (previously in Overlay.less).
      
      Bug: T216198
      Change-Id: Icce158581be7e29d6a1991815061cba634a56056
      6a44a004