Jhernandez (Joaquin Oltra Hernandez)
Senior Software Engineer, Reading Web Team

Projects (9)

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Saturday

  • Clear sailing ahead.

User Details

User Since
Oct 24 2014, 11:57 PM (160 w, 5 d)
Availability
Available
IRC Nick
joakino
LDAP User
Jhernandez
MediaWiki User
Unknown

Recent Activity

Yesterday

Jhernandez committed rMARVb269543d38a3: Fix: wiki and summary page title decoding (authored by Niedzielski).
Fix: wiki and summary page title decoding
Wed, Nov 22, 1:35 PM
Jhernandez added a comment to T177000: Create generic Error Page template.

Functionality merged, design implementation missing.

Wed, Nov 22, 1:30 PM · Design, Marvin
Jhernandez committed rMARV05ca82f58ad4: Update: capture wiki link clicks and navigation (authored by Niedzielski).
Update: capture wiki link clicks and navigation
Wed, Nov 22, 12:53 PM

Tue, Nov 21

Jhernandez committed rMARV4d13b5cb4a76: Update: show 404 page when content cannot be found (authored by Niedzielski).
Update: show 404 page when content cannot be found
Tue, Nov 21, 5:56 PM
Jhernandez closed T179829: Add images to File pages as Resolved.
Tue, Nov 21, 5:04 PM · Patch-For-Review, Marvin
Jhernandez closed T179829: Add images to File pages, a subtask of T176958: Display article content within article namespace, as Resolved.
Tue, Nov 21, 5:04 PM · Epic, Marvin

Mon, Nov 20

Jhernandez moved T180898: Font size is too small for wiki pages in Samsung galaxy j7 phone. from To Triage to Needs Analysis on the Readers-Web-Backlog board.
Mon, Nov 20, 11:46 AM · Readers-Web-Backlog, MobileFrontend
Jhernandez added a project to T180898: Font size is too small for wiki pages in Samsung galaxy j7 phone.: Readers-Web-Backlog.
Mon, Nov 20, 11:45 AM · Readers-Web-Backlog, MobileFrontend
Jhernandez added a comment to T180356: Popups EventLogging events occasionally invalid.

Sorry, I don't understand the question.

Mon, Nov 20, 10:29 AM · Readers-Web-Backlog, Page-Previews

Fri, Nov 17

Jhernandez closed T179826: Improve bundle size by replacing NamedModulesPlugin as Resolved.
Fri, Nov 17, 12:10 PM · Patch-For-Review, Marvin
Jhernandez closed T179826: Improve bundle size by replacing NamedModulesPlugin, a subtask of T177058: Set up performance tooling & dashboards, as Resolved.
Fri, Nov 17, 12:10 PM · Performance-Team (Radar), Epic, Marvin
Jhernandez added a comment to T180623: [Performance] Use an img for loading the wordmark.

Did some amends on the last patch, have a look @Niedzielski and merge if you want

Fri, Nov 17, 12:07 PM · Patch-For-Review, Marvin

Thu, Nov 16

Jhernandez changed the status of T179305: Add favicon, a subtask of T177367: Initial version of Marvin user interface chrome, from Stalled to Open.
Thu, Nov 16, 6:44 PM · Patch-For-Review, Epic, Marvin
Jhernandez changed the status of T179305: Add favicon from Stalled to Open.
Thu, Nov 16, 6:44 PM · Patch-For-Review, Design, Marvin

Wed, Nov 15

Jhernandez added a comment to T179305: Add favicon.

Initially we should set just one, like with the logo, and we'll have to discuss later how we swap them when the language changes.

Wed, Nov 15, 1:19 PM · Patch-For-Review, Design, Marvin
Jhernandez added a comment to T179381: Add NPM dependency security audit CI job.

Makes sense

Wed, Nov 15, 1:16 PM · Technical-Debt, Marvin
Jhernandez added a comment to T180255: Tests broken under sinon 4.x.

I think most of the failures are due to the removal of the deprecated signature of stub stub(obj, 'meth', fn).

Wed, Nov 15, 12:43 PM · Readers-Web-Backlog, Page-Previews

Tue, Nov 14

Jhernandez updated the task description for T177057: Clarify production deployment requirements, outline and blockers.
Tue, Nov 14, 5:45 PM · Epic, Marvin
Jhernandez renamed T177057: Clarify production deployment requirements, outline and blockers from Set up production deployment to Clarify production deployment requirements, outline and blockers.
Tue, Nov 14, 5:36 PM · Epic, Marvin

Mon, Nov 13

Jhernandez awarded T176965: Upstream AnyComponent definition to preact a Burninate token.
Mon, Nov 13, 5:06 PM · Patch-For-Review, Marvin

Fri, Nov 10

Jhernandez created T180255: Tests broken under sinon 4.x.
Fri, Nov 10, 4:28 PM · Readers-Web-Backlog, Page-Previews
Jhernandez triaged T180219: Migrate stack.wmflabs.org to toolforge as Normal priority.
Fri, Nov 10, 12:58 PM · User-Jhernandez
Jhernandez moved T180219: Migrate stack.wmflabs.org to toolforge from Triage to Backlog on the User-Jhernandez board.
Fri, Nov 10, 12:57 PM · User-Jhernandez
Jhernandez created T180219: Migrate stack.wmflabs.org to toolforge.
Fri, Nov 10, 12:55 PM · User-Jhernandez

Thu, Nov 9

Jhernandez updated the task description for T178403: Gather information about devices, network speeds to run tests about.
Thu, Nov 9, 11:19 AM · Marvin

Wed, Nov 8

Jhernandez added a comment to T180054: Call to a member function isMainPage() on a non-object (null).

Is this a duplicate of T179833: PHP Exception: Call to a member function isMainPage() on a non-object (null)?

Wed, Nov 8, 6:33 PM · Readers-Web-Backlog, MinervaNeue
Jhernandez added a comment to T180049: Anonymous fully-loaded and first paint (moving medium) page time significantly increased on 3G Barack Obama en.wiki on November 3rd, 06:00.

Maybe related to the banner with an image?:

Wed, Nov 8, 6:32 PM · Readers-Web-Backlog (Tracking), Fundraising-Backlog, Performance-Team, Regression, Performance
Jhernandez added a comment to T169369: Change font size controls to use small, regular, large and extra large buckets.

For measuring the CSS & JS sizes you can use mw.loader.inspect, or in an incognito window open the network tab and see the total sizes at the bottom after loading the mentioned page (something like https://m.mediawiki.org/wiki/Special:MobileOptions?mobileaction=beta)

Wed, Nov 8, 5:38 PM · MobileFrontend, Patch-For-Review, Readers-Web-Kanban-Board, Readers-Web-Backlog
Jhernandez updated the task description for T169369: Change font size controls to use small, regular, large and extra large buckets.
Wed, Nov 8, 5:29 PM · MobileFrontend, Patch-For-Review, Readers-Web-Kanban-Board, Readers-Web-Backlog
Jhernandez updated the task description for T169369: Change font size controls to use small, regular, large and extra large buckets.
Wed, Nov 8, 5:28 PM · MobileFrontend, Patch-For-Review, Readers-Web-Kanban-Board, Readers-Web-Backlog

Tue, Nov 7

Jhernandez committed rMARVaa5571b654f6: Chore: refactor page modules (authored by Niedzielski).
Chore: refactor page modules
Tue, Nov 7, 5:42 PM
Jhernandez updated the description for M232: MediaWiki ResourceLoader Overview.
Tue, Nov 7, 11:48 AM
Jhernandez updated the description for M232: MediaWiki ResourceLoader Overview.
Tue, Nov 7, 11:44 AM

Mon, Nov 6

Jhernandez updated the description for M232: MediaWiki ResourceLoader Overview.
Mon, Nov 6, 6:37 PM
Jhernandez created M232: MediaWiki ResourceLoader Overview.
Mon, Nov 6, 6:34 PM
Jhernandez updated the task description for T179833: PHP Exception: Call to a member function isMainPage() on a non-object (null).
Mon, Nov 6, 4:40 PM · MW-1.31-release-notes (WMF-deploy-2017-11-14 (1.31.0-wmf.8)), Patch-For-Review, Readers-Web-Backlog (Tracking), Wikimedia-log-errors, MinervaNeue
Jhernandez created T179833: PHP Exception: Call to a member function isMainPage() on a non-object (null).
Mon, Nov 6, 4:39 PM · MW-1.31-release-notes (WMF-deploy-2017-11-14 (1.31.0-wmf.8)), Patch-For-Review, Readers-Web-Backlog (Tracking), Wikimedia-log-errors, MinervaNeue
Jhernandez moved T179598: New responsive design for old Navboxes from Needs triage to Triaged on the Mobile board.
Mon, Nov 6, 4:32 PM · MediaWiki-General-or-Unknown, Mobile
Jhernandez moved T179808: mobile view: first column in infoboxes is too wide from Needs triage to Triaged on the Mobile board.
Mon, Nov 6, 4:31 PM · Readers-Web-Backlog, MinervaNeue, MobileFrontend, Mobile
Jhernandez added a project to T179808: mobile view: first column in infoboxes is too wide: Readers-Web-Backlog.
Mon, Nov 6, 4:26 PM · Readers-Web-Backlog, MinervaNeue, MobileFrontend, Mobile

Thu, Nov 2

Jhernandez assigned T177004: Page switching interaction to Nirzar.

We're missing the design specs so let's assign to @Nirzar

Thu, Nov 2, 1:48 PM · Epic, Design, Marvin
Jhernandez moved T172923: [Placeholder] Stephen and Joaquin are working on #marvin from Needs More Work to Ready for Signoff on the Readers-Web-Kanban-Board board.
Thu, Nov 2, 1:46 PM · Category, Readers-Web-Kanban-Board, Marvin
Jhernandez added a comment to T176959: Display proper section collapsing within article namespace.

@Fjalapeno @bearND this tix is/was mostly about the UI part for us, but it has a lot of conversation. I'm happy to make a new one for us and unlink it this one from our board if you want to preserve the convo here! Let me know what you decide.

Thu, Nov 2, 1:41 PM · Marvin, Epic
Jhernandez added a comment to T177236: Export pages as data implementing PageModule.

@Niedzielski It is what it is if we want an explicit PageModule typing. It seems to me we should use default exports instead of a named page export, so:

Thu, Nov 2, 1:38 PM · Patch-For-Review, Marvin
Jhernandez created T179574: List editing exception: TypeError: Cannot read property 'getSurface' of null.
Thu, Nov 2, 1:07 PM · VisualEditor-ContentEditable, Patch-For-Review, VisualEditor

Wed, Nov 1

Jhernandez changed the status of T178407: Define navigation scenarios to test and measure, a subtask of T177058: Set up performance tooling & dashboards, from Stalled to Open.
Wed, Nov 1, 12:47 PM · Performance-Team (Radar), Epic, Marvin
Jhernandez changed the status of T178407: Define navigation scenarios to test and measure from Stalled to Open.
Wed, Nov 1, 12:47 PM · Marvin
Jhernandez changed the status of T178403: Gather information about devices, network speeds to run tests about from Stalled to Open.
Wed, Nov 1, 12:47 PM · Marvin
Jhernandez changed the status of T178403: Gather information about devices, network speeds to run tests about, a subtask of T177058: Set up performance tooling & dashboards, from Stalled to Open.
Wed, Nov 1, 12:47 PM · Performance-Team (Radar), Epic, Marvin
Jhernandez added a comment to T178403: Gather information about devices, network speeds to run tests about.

Added the details from that article to the description grouped by source

Wed, Nov 1, 11:39 AM · Marvin
Jhernandez updated the task description for T178403: Gather information about devices, network speeds to run tests about.
Wed, Nov 1, 11:39 AM · Marvin
Jhernandez assigned T178407: Define navigation scenarios to test and measure to ovasileva.
Wed, Nov 1, 10:59 AM · Marvin
Jhernandez assigned T178403: Gather information about devices, network speeds to run tests about to ovasileva.
Wed, Nov 1, 10:59 AM · Marvin
Jhernandez added a comment to T178403: Gather information about devices, network speeds to run tests about.

This https://infrequently.org/2017/10/can-you-afford-it-real-world-web-performance-budgets/ has some interesting data, and it mentions the Moto G4 for last year and the Moto G5 for this year:

Wed, Nov 1, 10:57 AM · Marvin
Jhernandez added a comment to T176959: Display proper section collapsing within article namespace.

Quick note that section collapsing on mobile viewports is what we do now on mobile web, but it is very possible that the model will change in the future, based on product requirements from user tests. We have already seen that the apps are using a different TOC and section UX so that may be the direction that mobile web is asked to take in the future.

Wed, Nov 1, 10:51 AM · Marvin, Epic

Tue, Oct 31

Jhernandez added a comment to T179425: mediawiki/extensions/Popups runs qunit in normal "npm test".

It seems like QUnit has nothing to do with the problem.

Tue, Oct 31, 6:21 PM · Readers-Web-Backlog (Tracking), Unplanned-Sprint-Work, Patch-For-Review, Page-Previews, Continuous-Integration-Config
Jhernandez added a comment to T179381: Add NPM dependency security audit CI job.

Should this be added to npm test to run on CI? Why does it need to be a new job?

Tue, Oct 31, 1:33 PM · Technical-Debt, Marvin
Jhernandez added a comment to T176959: Display proper section collapsing within article namespace.

Lots of good points. Wrapping the content is a nice convenience for reads but we could maybe get away by other means if it is a big problem.

Tue, Oct 31, 1:16 PM · Marvin, Epic

Oct 20 2017

Jhernandez added a comment to T178615: Add query parameter typing.

Thanks for the through explanation.

Oct 20 2017, 9:34 AM · Marvin
Jhernandez awarded T178616: Move wiki revision path parameter to query parameter a Like token.
Oct 20 2017, 9:14 AM · Marvin
Jhernandez added a comment to T178617: Add query parameter parser.

Some info I've found:

Oct 20 2017, 9:13 AM · Marvin

Oct 18 2017

Jhernandez added a parent task for T177681: Page URL should update on redirect: T173317: Initial version of article rendering.
Oct 18 2017, 11:13 AM · Patch-For-Review, Marvin
Jhernandez added a subtask for T173317: Initial version of article rendering: T177681: Page URL should update on redirect.
Oct 18 2017, 11:13 AM · Marvin, Epic

Oct 17 2017

Jhernandez added a comment to T176211: Page Previews could load less JS on pageload.

We should have good interaction data to know how quickly users interact with links from the event logging no?

Oct 17 2017, 5:37 PM · Patch-For-Review, Performance-Team (Radar), Readers-Web-Backlog, Page-Previews
Jhernandez placed T178407: Define navigation scenarios to test and measure up for grabs.
Oct 17 2017, 4:58 PM · Marvin
Jhernandez created T178407: Define navigation scenarios to test and measure.
Oct 17 2017, 4:57 PM · Marvin
Jhernandez placed T178403: Gather information about devices, network speeds to run tests about up for grabs.
Oct 17 2017, 4:46 PM · Marvin
Jhernandez added a comment to T178403: Gather information about devices, network speeds to run tests about.

It may be helpful to consider the personas and information from the new readers reports, and other profiles from the developed countries to have a comprehensive list.

Oct 17 2017, 4:46 PM · Marvin
Jhernandez created T178403: Gather information about devices, network speeds to run tests about.
Oct 17 2017, 4:45 PM · Marvin

Oct 16 2017

Jhernandez added a comment to T177681: Page URL should update on redirect.

Some AC

Oct 16 2017, 3:37 PM · Patch-For-Review, Marvin
Jhernandez closed T177368: Hide menu and and search icons as Resolved.
Oct 16 2017, 2:59 PM · Patch-For-Review, Marvin
Jhernandez closed T177368: Hide menu and and search icons, a subtask of T177367: Initial version of Marvin user interface chrome, as Resolved.
Oct 16 2017, 2:59 PM · Patch-For-Review, Epic, Marvin

Oct 6 2017

Jhernandez added a comment to T177464: Submit an RFC and discuss it with TechCom.

I find it helpful to distinguish between deep architecture/technology decisions which are hard to reverse, and feature/product/UI decisions that are relatively easy.

Oct 6 2017, 6:59 AM · Marvin

Oct 5 2017

Jhernandez added a comment to T177464: Submit an RFC and discuss it with TechCom.

Thanks for the insights @ksmith

Oct 5 2017, 6:10 PM · Marvin
Jhernandez added a comment to T177058: Set up performance tooling & dashboards.

@Peter thanks for commenting! We just ended up planning the epics for the quarter so we're ready now to start with this.

Oct 5 2017, 10:53 AM · Performance-Team (Radar), Epic, Marvin
Jhernandez updated the task description for T177464: Submit an RFC and discuss it with TechCom.
Oct 5 2017, 10:22 AM · Marvin
Jhernandez added a comment to T177057: Clarify production deployment requirements, outline and blockers.

What did I miss here?

Oct 5 2017, 10:16 AM · Epic, Marvin
Jhernandez added a comment to T177464: Submit an RFC and discuss it with TechCom.

Some related links to add to the RFC:

Oct 5 2017, 10:10 AM · Marvin

Oct 4 2017

Jhernandez added a comment to T177344: Popups use of sourceMappingURL breaks production debugging.

As mentioned in the description, removing the @nomin banner could also work well giving the sourcemap comment on debug=true and having jsmin remove it on debug=false.

Oct 4 2017, 5:49 PM · MW-1.31-release-notes (WMF-deploy-2017-10-17 (1.31.0-wmf.4)), Patch-For-Review, Easy, Readers-Web-Kanban-Board, Readers-Web-Backlog, Performance-Team (Radar), Page-Previews
Jhernandez added a comment to T177344: Popups use of sourceMappingURL breaks production debugging.

For reference, this involves unsetting https://github.com/wikimedia/mediawiki-extensions-Popups/blob/master/webpack.config.js#L35 when compiling for production.

Oct 4 2017, 5:45 PM · MW-1.31-release-notes (WMF-deploy-2017-10-17 (1.31.0-wmf.4)), Patch-For-Review, Easy, Readers-Web-Kanban-Board, Readers-Web-Backlog, Performance-Team (Radar), Page-Previews
Jhernandez awarded T177210: Security review of Marvin a Like token.
Oct 4 2017, 11:48 AM · Security-Reviews, Marvin
Jhernandez added a comment to T167699: [EPIC] Enable JS error reporting in the mobile website.

Thanks for the feedback @Tgr and @matmarex.

Oct 4 2017, 11:47 AM · Epic, Readers-Web-Backlog, MobileFrontend
Jhernandez added a comment to T150377: Use 'outside' instead of 'Inside' for list-style in Mobile version so that long line bullet list items are more scannable.

@Jdlrobson To be clear I don't think we should use padding-start, since it is not well supported. We should use padding-left and which will flip to padding right on rtl languages. padding-left + janus is like the polyfill for padding-start. It doesn't seem like a hack to me, it is like a polyfill until we have padding-start supported everywhere.

Oct 4 2017, 11:42 AM · Unplanned-Sprint-Work, Readers-Web-Kanban-Board, MW-1.31-release-notes (WMF-deploy-2017-11-07 (1.31.0-wmf.7)), User-Jdlrobson, MinervaNeue, Readers-Web-Backlog, Design, WMF-Design
Jhernandez added a comment to T176958: Display article content within article namespace.

See T177367: Initial version of Marvin user interface chrome for the chrome epic

Oct 4 2017, 11:32 AM · Epic, Marvin
Jhernandez renamed T177368: Hide menu and and search icons from Remove menu and and search icons to Hide menu and and search icons.
Oct 4 2017, 11:24 AM · Patch-For-Review, Marvin
Jhernandez added a comment to T177367: Initial version of Marvin user interface chrome.

👍 Good catch.

Oct 4 2017, 11:22 AM · Patch-For-Review, Epic, Marvin

Oct 3 2017

Jhernandez updated subscribers of T173317: Initial version of article rendering.

@Niedzielski would you mind adding the patches to the subtasks of the Epic? For example rMARV23555ba64ed9: New: add dummy wiki page could be a fit for T173324: Add page route and rendering.

Oct 3 2017, 5:35 PM · Marvin, Epic
Jhernandez added a comment to T150377: Use 'outside' instead of 'Inside' for list-style in Mobile version so that long line bullet list items are more scannable.

It is not faking, it is how browsers do it by default! 😃

Oct 3 2017, 5:32 PM · Unplanned-Sprint-Work, Readers-Web-Kanban-Board, MW-1.31-release-notes (WMF-deploy-2017-11-07 (1.31.0-wmf.7)), User-Jdlrobson, MinervaNeue, Readers-Web-Backlog, Design, WMF-Design
Jhernandez moved T177010: Improve style guide structure from Next Q Epics (2018 Q3) to Current Epics (2017-Q2) on the Marvin board.
Oct 3 2017, 5:26 PM · Epic, Design, Marvin
Jhernandez moved T177058: Set up performance tooling & dashboards from Next Q Epics (2018 Q3) to Current Epics (2017-Q2) on the Marvin board.
Oct 3 2017, 5:26 PM · Performance-Team (Radar), Epic, Marvin
Jhernandez moved T177057: Clarify production deployment requirements, outline and blockers from Next Q Epics (2018 Q3) to Current Epics (2017-Q2) on the Marvin board.
Oct 3 2017, 5:26 PM · Epic, Marvin
Jhernandez moved T175806: Implement grid & responsive system from Next Q Epics (2018 Q3) to Current Epics (2017-Q2) on the Marvin board.
Oct 3 2017, 5:26 PM · Epic, Design, Marvin
Jhernandez moved T177004: Page switching interaction from Next Q Epics (2018 Q3) to Current Epics (2017-Q2) on the Marvin board.
Oct 3 2017, 5:26 PM · Epic, Design, Marvin
Jhernandez moved T176980: Article namespace must display last edited information from Next Q Epics (2018 Q3) to Current Epics (2017-Q2) on the Marvin board.
Oct 3 2017, 5:26 PM · Epic, Marvin
Jhernandez moved T176959: Display proper section collapsing within article namespace from Next Q Epics (2018 Q3) to Current Epics (2017-Q2) on the Marvin board.
Oct 3 2017, 5:26 PM · Marvin, Epic
Jhernandez moved T176958: Display article content within article namespace from Next Q Epics (2018 Q3) to Current Epics (2017-Q2) on the Marvin board.
Oct 3 2017, 5:26 PM · Epic, Marvin
Jhernandez claimed T177058: Set up performance tooling & dashboards.
Oct 3 2017, 3:42 PM · Performance-Team (Radar), Epic, Marvin
Jhernandez renamed T173328: Add content page section component and render all sections from Add page section component and render all sections to Add content page section component and render all sections.
Oct 3 2017, 3:26 PM · Patch-For-Review, Marvin
Jhernandez renamed T173326: Extract page title component from /summary and use it on /wiki from Add page title component and render it to Extract page title component from /summary and use it on /wiki.
Oct 3 2017, 3:24 PM · Patch-For-Review, Marvin
Jhernandez updated the task description for T173330: Add content styles.
Oct 3 2017, 3:22 PM · Patch-For-Review, Marvin