A dungeon should take about half an hour to 45 minutes, that is reasonable. What did we do to Maw of Souls because it was by far the fastest dungeon to spam run for AP? Nerfed its yield. Same happened to dungeons. Except instead of nerfing the XP, they adjusted the clear speed to justify the experience a bit more. Is it perfect? Nah, a lot of dungeons aren't even that long with updates. At most a clear takes 5 mins more.
Some people have made arguments that quest blues can sometimes be more powerful than heirlooms, especially with sockets. I'm not convinced. The simple fact that you never have to even look at quest rewards will save a non-insignificant amount of time over the course of leveling. Also of consideration is the value of enchanted gear never having to be re-enchanted.
While leveling, try to make use of the new Battle of Azeroth experience gem, Insightful Rubellite, which gives you +5% experience. Equipping one of these requires a socket in your gear, so try to hold on to a piece of Legion gear with a socket or acquire one of the Legion Legendary rings since every class has one. Each of the rings includes a socket. You can acquire one of these rings rather easily by doing World Quests in Legion zones and acquiring 1,000 Wakening Essences that can be used to purchase a legendary ring in the Legion version of Dalaran.
Warfronts are larger, 20-man instances that take inspiration from the Warcraft strategy games. Players are deployed on the Warfront to collect resources, destroy enemy troops, construct buildings, and eventually launch an assault to take over the enemy base. These battles take 20 to 40 minutes to complete and, unlike Island Expeditions, don’t offer a PvP mode.
Excess Potion of Accelerated Learning: Acquired in your Garrison (part of the Warlords of Draenor expansion), this gives you a 20-percent bonus to experience from killing monsters and finishing quests. It lasts one hour, but you can buy it multiple times and use it back-to-back. Alliance players buy it from Sergeant Crowler, horde players buy it from Sergeant Grimjaw. Does not work above level 99.

Cooking is a secondary crafting profession, which is available in addition to your two main professions. It takes various food crafting materials such as meats and fish and turn them into consumable food. The consumables give statbuffs and are used by raiders and other players seeking to maximize performance. It is a great money maker and you should have it on your main.

Update: The reason for less experience from gathering is due to how Blizzard revamped professions and broke them down by expansion. In order to get full exp from gathering you'll simply need to find a profession trainer for the relevant expansion. (WoD you just mine or herb the first one ypu come across and the game gives you a scroll for learning WoD gathering).
That lack of storytelling agency may soon change with Patch 8.1, Tides of Vengeance, which is coming to Battle for Azeroth soon. While there are plenty of fun gameplay changes that should make the expansion better, one of the most interesting additions is that of choice. In an upcoming quest, players will be able to choose whether they want to support Saurfang or Sylvanas — essentially honor or evil — in the struggle for Horde warchief.
YouTuber Oldbess recently uploaded a video about this cool method.  Generally completing the scenario "Deaths of Chromie" will allow you to pick up the item "Time-Lost Wallet" in excess.  You can get it from just killing the mobs or actually completing the scenario in full.  By then end you will have dozens of time lost wallets.  Each wallet awards 50g AND a rare chance at getting the ]bronze proto-whelp[/U] pet.  In the video, this spot was VERY hard for to pull off as it requires a high item level.  With an 809 item level I struggled all throughout the scenario and could not make this spot profitable for the time invested.  The good news is, if you can get it done, the gold you make will be entirely liquidated – the rewards are 50g raw for each wallet turn in.  I suggest you watch it on Oldbess's channel for full details.
If i start on a new realms, I am farming a lot of transmog from classic and Burning Crusade Dungeons and Raids. In the beginning, i start to post them if they have at least a DBRegionMarketAvg (Average value of an item on all EU realms of the last 14 days) of 500 Gold. All the other items get vendored. The time you hit a stock of 1000 Items, my value treshold raises to around 2.500 Gold. All the items below that value get vendored aswell. Newly farmed items that are above these treshold will be listed. Why? If I start over somewhere, I want to get a certain amount of gold quickly. Cheap items are great for that. Later on, to save time, Quality is better than quantity. As mentioned above, re-posting the items on the auction house takes quite a bit of time. At this point, I can recommend you the TradeSkillMaster AddOn, it makes your auction house work really a hell of a lot easier.